Page MenuHomeFreeBSD

security/linux-c6-pam and security/linux-c6-audit-libs: New ports
ClosedPublic

Authored by arrowd on Oct 15 2018, 8:09 AM.
Tags
None
Referenced Files
F81599801: D17561.id.diff
Thu, Apr 18, 6:23 PM
Unknown Object (File)
Thu, Apr 18, 4:17 AM
Unknown Object (File)
Wed, Apr 17, 1:42 AM
Unknown Object (File)
Tue, Apr 16, 12:15 AM
Unknown Object (File)
Thu, Apr 11, 11:10 AM
Unknown Object (File)
Thu, Apr 11, 9:29 AM
Unknown Object (File)
Thu, Apr 11, 9:24 AM
Unknown Object (File)
Thu, Apr 11, 8:39 AM
Subscribers

Details

Summary

These libraries are required to run recent MATLAB under Linuxulator.
I haven't added pkg-plist.i386 files, as I don't have 32-bit VM by hand.

Test Plan

make install and ran MATLAB

Diff Detail

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

Event Timeline

Pinged emulation@ review this.

security/linux-c6-audit-libs/Makefile
10 ↗(On Diff #49152)

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

security/linux-c6-pam/Makefile
10 ↗(On Diff #49152)

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

arrowd added inline comments.
security/linux-c6-audit-libs/Makefile
10 ↗(On Diff #49152)

I copied that from existing security/linux-c6-{gnutls,cyrus-sasl2}. Are these wrong too?

You need to hook the new ports into the build via security/Makefile

security/linux-c6-audit-libs/Makefile
10 ↗(On Diff #49152)

Yes :)

arrowd marked 2 inline comments as done.

Address comments.

Added new ports to category Makefile.

You can create the i386 plist by using poudriere :)

poudriere jail -c -j104amd32 -ai386...

===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: etc/libaudit.conf
Error: Orphaned: usr/lib/libaudit.so.1
Error: Orphaned: usr/lib/libaudit.so.1.0.0
Error: Orphaned: usr/lib/libauparse.so.0
Error: Orphaned: usr/lib/libauparse.so.0.0.0
Error: Orphaned: usr/share/man/man5/libaudit.conf.5.gz

and so on...

Please add that plist too.

This revision now requires changes to proceed.Oct 27 2018, 12:29 PM

Add pkg-plist's for i386 arch.

security/linux-c6-pam/pkg-plist.amd64
6 ↗(On Diff #49683)

^ I personally don't like subs liek these, that are partial. But I think in that case we might let it go by.

Looks good to me, once you add the '\n' to the last lines of the pkg-descrs

security/linux-c6-pam/pkg-descr
7 ↗(On Diff #49683)

^ add the new line chars on the last line.

This revision is now accepted and ready to land.Oct 28 2018, 9:13 PM
This revision was automatically updated to reflect the committed changes.