Page MenuHomeFreeBSD

net-im/py-slixmpp: Add missing RUN_DEPENDS and fix some other issues
ClosedPublic

Authored by 0mp on Dec 29 2018, 8:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 2:04 PM
Unknown Object (File)
Tue, Apr 16, 9:54 PM
Unknown Object (File)
Tue, Apr 16, 9:54 PM
Unknown Object (File)
Tue, Apr 16, 9:54 PM
Unknown Object (File)
Tue, Apr 16, 9:37 PM
Unknown Object (File)
Jan 15 2024, 3:55 PM
Unknown Object (File)
Jan 8 2024, 2:05 AM
Unknown Object (File)
Dec 20 2023, 2:56 AM
Subscribers

Details

Summary

Changes:

  • Switch MASTER_SITES to CHEESESHOP as it ought to be the default for Python software.
  • Adjust COMMENT to match the short description from setup.py.
  • Add missing RUN_DEPENDS.
  • Standardize test invocation.
  • Update WWW to point to project's homepage instead of the Git repository.

Thanks to @koobs for reviewing my commit and telling me how I can improve. :)

Test Plan
  • portlint
  • poudriere testport 11.2 on amd64 with the test suite

Diff Detail

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

Event Timeline

Reviewed_by: koobs
Approved by: koobs

net-im/py-slixmpp/Makefile
23 โ†—(On Diff #52403)

The pkgconfig bit may not be necessary, I just wanted to mention it in review email in case it was relevant. pkg-config is one of those mechanisms that is often attempted/autodetected in build systems, so builds can be non deterministic if ports logic is not bang on.

There's plenty of systems that have pkg-config installed when building ports, unrelated to the port build in question. Clean-room environments like poudriere wont pick this up without specific testing.

One thing you can try is to unconditionally add the dependency on pkg-config and see if/how the build/package changes with/without it installed.

This revision is now accepted and ready to land.Dec 30 2018, 2:00 AM
This revision was automatically updated to reflect the committed changes.
0mp marked an inline comment as done.Jan 8 2019, 1:32 PM
0mp added inline comments.
net-im/py-slixmpp/Makefile
23 โ†—(On Diff #52403)

I left pkgconfig there. Upstream requires it for building, however the package builds just fine without it. I'll leave it here for the time being.