Page MenuHomeFreeBSD

security/hardening-check: Change MASTER_SITES to DEBIAN pool and remove SHEBANG_LANG
ClosedPublic

Authored by ultima on Jul 9 2017, 12:08 AM.
Tags
None
Referenced Files
Unknown Object (File)
May 14 2024, 12:14 AM
Unknown Object (File)
May 7 2024, 5:08 AM
Unknown Object (File)
May 4 2024, 6:03 PM
Unknown Object (File)
May 4 2024, 6:03 PM
Unknown Object (File)
May 4 2024, 6:03 PM
Unknown Object (File)
May 4 2024, 6:03 PM
Unknown Object (File)
May 4 2024, 5:52 PM
Unknown Object (File)
May 1 2024, 1:05 AM
Subscribers

Details

Summary
  • Changed MASTER_SITES to DEBIAN pool
  • Removed SHEBANG_LANG, shebandfix already includes this for perl

PR\: 220569
Reported by\: matt
Reviewed by\: lifanov (mentor), matthew (mentor), Jov <amutu@amutu.com> (maintainer)
Approved by\: lifanov (mentor), matthew (mentor), Jov <amutu@amutu.com> (maintainer)
Differential Revision\: https://reviews.freebsd.org/DXXXXX

Test Plan

portlint:
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy.
WARN: Makefile: using hyphen in PORTNAME. consider using PKGNAMEPREFIX and/or PKGNAMESUFFIX.
0 fatal errors and 2 warnings found.

poudriere:
103i386
103amd64
110i386
110amd64
12i386
12amd64

Diff Detail

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

Event Timeline

ultima retitled this revision from security/hardening-check: Change MASTER_SITES to DEBIAN pool and remove SHEBANG_LANGsecurity/hardening-check: Change MASTER_SITES to DEBIAN pool and remove SHEBANG_LANG to security/hardening-check: Change MASTER_SITES to DEBIAN pool and remove SHEBANG_LANG.Jul 9 2017, 12:09 AM
This revision is now accepted and ready to land.Jul 9 2017, 2:12 AM
This revision was automatically updated to reflect the committed changes.
head/security/hardening-check/Makefile
6–8

This is:

​MASTER_SITES=        DEBIAN/pool/main/h/hardening-wrapper

Never use MASTER_SITE_SUBDIR, I spent enough time removing all occurrences I could, do not add them. (Side note, the porter's handbook tells you exactly that too)

head/security/hardening-check/Makefile
6–8

Sorry about that, I didn't realize it was deprecated.

head/security/hardening-check/Makefile
6–8

Mmmm, deprecated is maybe a bit too strong, discourraged is more like it. Most of the time, you do not need to use it, and should refrain from.
The most frequent use case is when fetching from CPAN.
There may also be some rare cases where using it is makes it simpler and shorter, but those cases few.

head/security/hardening-check/Makefile
6–8

-MASTER_SITES= DEBIAN
-MASTER_SITE_SUBDIR= pool/main/h/hardening-wrapper
+MASTER_SITES= DEBIAN/pool/main/h/hardening-wrapper

I need approval to commit.

matthew accepted this revision.

Approved.

This revision is now accepted and ready to land.Jul 14 2017, 7:34 AM
This revision was automatically updated to reflect the committed changes.