Page MenuHomeFreeBSD

New port: cad/sumo: Traffic simulation suite for road vehicles, public transportation, etc
ClosedPublic

Authored by yuri on Nov 26 2017, 10:13 AM.
Tags
None
Referenced Files
F105830874: D13248.id36156.diff
Sat, Dec 21, 9:22 AM
Unknown Object (File)
Fri, Dec 13, 12:40 PM
Unknown Object (File)
Wed, Dec 11, 8:56 PM
Unknown Object (File)
Mon, Dec 9, 10:05 AM
Unknown Object (File)
Sun, Dec 8, 2:03 AM
Unknown Object (File)
Sun, Dec 8, 12:15 AM
Unknown Object (File)
Wed, Dec 4, 6:24 PM
Unknown Object (File)
Nov 6 2024, 1:18 PM
Subscribers

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 13150
Build 13396: arc lint + arc unit

Event Timeline

cad/sumo/Makefile
8

The -doc- distfile should probably be in a DOCS_DISTFILES variable.

44–45

This is probably wrong. To install files in STAGEDIR, you must use the INSTALL_* or COPYTREE_* macros so that the rights are correct.

yuri marked 2 inline comments as done.Nov 28 2017, 7:13 AM
cad/sumo/Makefile
8

This is an option helper, it happens way too early.

43

This should probably be find -d so that recursively empty directories are detected.

44

This should probably be done with USES=shebangfix.

yuri marked 3 inline comments as done.

Moved DOCS_DISTFILES to option helpers section.

cad/sumo/Makefile
44

DOCS_SHEBANG_FILES doesn't work. So I will have to put SHEBANG_FILES= docs/tutorial/*/*.py into the option helpers section.

Do you still want me to do this?

devel/phpbt/Makefile has a conditional USES=shebangfix but SHEBANG_FILES is still used. But shouldn't all such USES arguments be made conditional too?

cad/sumo/Makefile
44

Please see the * (Any variable) bit of the list in 5.13.3. Options Helpers to learn how to deal with variables only needed by options.

yuri marked 2 inline comments as done.Nov 29 2017, 5:53 PM
yuri added inline comments.
cad/sumo/Makefile
44

Thanks!

yuri marked 2 inline comments as done.Nov 29 2017, 5:53 PM
cad/sumo/distinfo
2

misses the sumo-doc distfile

yuri marked an inline comment as done.Nov 29 2017, 9:10 PM
yuri added inline comments.
cad/sumo/distinfo
2

DOCS_DISTFILES is broken. It overwrites DISTFILES instead of adding to it.

cad/sumo/distinfo
2

Well, what I told you to do would have worked, but you changed the DISTFILES to DISTNAME, and of course, that broke it.

yuri marked an inline comment as done.

Fixed distfiles.

cad/sumo/Makefile
8

I am not sure referencing the PR is a good idea, as it is not a bug, it is documented that way, and it cannot work any other way.

yuri marked 3 inline comments as done.

Removed bug# comment.

cad/sumo/Makefile
8

Removed.

yuri marked an inline comment as done.Dec 1 2017, 4:56 PM

@tcberner Tobias, does it look like everything is ok?

Unfortunately not. You need to fix the plist for the debug builds :) -- the binaries get a "D" suffix :)

===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: bin/TraCITestClientD
Error: Orphaned: bin/activitygenD
[...]
===> Checking for items in pkg-plist which are not in STAGEDIR
Error: Missing: bin/TraCITestClient
Error: Missing: bin/activitygen
[...]
===> Error: Plist issues found.
*** Error code 1

I suggest using some PLIST_SUB %%DEBUG_SUFFIX%% or soemthing :)

I removed the DEBUG option.
It doesn't need to be installed when debugged - it's just a set of executables. Anybody can still enable debug locally.

cad/sumo/Makefile
45

^ why is this comment still here?

yuri marked an inline comment as done.Dec 3 2017, 7:16 PM
This revision is now accepted and ready to land.Dec 3 2017, 7:16 PM
This revision was automatically updated to reflect the committed changes.