Page MenuHomeFreeBSD

devel/picojson: New port
ClosedPublic

Authored by james.wright_digital-chaos.com on Apr 13 2019, 3:14 PM.
Referenced Files
F80157471: D19899.id.diff
Thu, Mar 28, 4:41 PM
Unknown Object (File)
Feb 20 2024, 2:46 PM
Unknown Object (File)
Jan 16 2024, 2:29 AM
Unknown Object (File)
Dec 23 2023, 11:56 PM
Unknown Object (File)
Dec 23 2023, 2:55 AM
Unknown Object (File)
Jul 2 2023, 8:30 AM
Unknown Object (File)
Jun 29 2023, 7:23 AM
Unknown Object (File)
May 21 2023, 1:02 AM
Subscribers

Details

Summary

Created a new port: devel/picojson

Bugzilla:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=237262

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

tcberner added a subscriber: tcberner.

The port needs to be hooked into devel/Makefile

devel/picojson/Makefile
9

^ don't add new ports without a maintainer.

This revision now requires changes to proceed.Apr 13 2019, 3:26 PM

Added picojson subdirectory to devel/Makefile
Added myself as maintainer of this port

tobik requested changes to this revision.Apr 14 2019, 6:07 AM
tobik added a subscriber: tobik.
tobik added inline comments.
devel/picojson/Makefile
15–16

Wrong place in the Makefile. See 15. Order of Variables in Port Makefiles.

Move the options block below ALL_TARGET.

21

Looking at picojson's Makefile [1] it does not seem to build anything and this will cause
the tests to run during a normal package build which we normally do not do, so maybe set NO_BUILD=yes and TEST_TARGET=check instead.

[1] https://github.com/kazuho/picojson/blob/v1.3.0/Makefile

This revision now requires changes to proceed.Apr 14 2019, 6:07 AM
james.wright_digital-chaos.com marked an inline comment as done.

Updated Makefile as suggested by tobik;
Removed ALL_TARGET=
Added NO_BUILD= yes
Added TEST_TARGET= check
Moved 'OPTIONS' variables down for correct ordering

Updated pkg-desc;
Shortened a line that was > 80 chars

james.wright_digital-chaos.com added inline comments.
devel/picojson/Makefile
15–16

Thanks for the feedback, I didn't notice the wrong ordering as the portlint run was clean!

tobik added inline comments.
devel/picojson/pkg-descr
6

typo: frendly -> friendly

Are there any further changes that this port needs, or can it be merged?

This revision is now accepted and ready to land.Apr 19 2019, 7:19 AM
This revision was automatically updated to reflect the committed changes.