Page MenuHomeFreeBSD

New port: science/mcxtrace: Monte Carlo X-ray tracing package
ClosedPublic

Authored by yuri on Nov 23 2017, 10:29 PM.
Tags
None
Referenced Files
F171657130: D13216.id35864.diff
Sat, Sep 12, 11:54 AM
F171643719: D13216.id35672.diff
Sat, Sep 12, 9:47 AM
F171624233: D13216.id35713.diff
Sat, Sep 12, 6:33 AM
F171586068: D13216.id.diff
Fri, Sep 11, 10:31 PM
Unknown Object (File)
Thu, Sep 10, 6:49 AM
Unknown Object (File)
Tue, Sep 8, 3:30 PM
Unknown Object (File)
Mon, Sep 7, 1:45 AM
Unknown Object (File)
Sat, Aug 29, 11:46 PM
Subscribers

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

science/mcxtrace/Makefile
22–25 ↗(On Diff #35672)

If the man pages do not need some extra dependency to be built, they must always be installed, without any option.

mat requested changes to this revision.Nov 24 2017, 6:24 PM
This revision now requires changes to proceed.Nov 24 2017, 6:24 PM

Removed the MANOPAGES option.

science/mcxtrace/pkg-plist
12 ↗(On Diff #35713)

^ why are the c-files installed too?

yuri marked 2 inline comments as done.Nov 24 2017, 7:59 PM
yuri added inline comments.
science/mcxtrace/pkg-plist
12 ↗(On Diff #35713)

I forwarded this question to the original submitter.

yuri marked 2 inline comments as done.Nov 24 2017, 7:59 PM
science/mcxtrace/pkg-plist
12 ↗(On Diff #35713)

The original submitted confirmed that .c/.h are indeed needed due to the way how the program works.

yuri marked an inline comment as done.Nov 24 2017, 8:57 PM

That is one weird program then :D

Remember, that you still need the ok of the other persons here =)

Remember, that you still need the ok of the other persons here =)

Ok!

This is ready.
These .c/.h files are need for it to run.

This revision is now accepted and ready to land.Nov 27 2017, 1:31 PM
science/mcxtrace/Makefile
24 ↗(On Diff #35713)

This should be using ${RLN} instead.

This revision now requires review to proceed.Nov 27 2017, 5:22 PM
yuri marked an inline comment as done.Nov 27 2017, 5:23 PM
This revision was automatically updated to reflect the committed changes.