Page MenuHomeFreeBSD

Porters Handbook: Remove shar file instructions
AbandonedPublic

Authored by ndowens_yahoo.com on Aug 25 2018, 6:17 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 13 2023, 5:18 AM
Unknown Object (File)
Nov 22 2023, 3:09 PM
Unknown Object (File)
Nov 12 2023, 8:56 AM
Unknown Object (File)
Oct 12 2023, 3:38 AM
Unknown Object (File)
Aug 15 2023, 7:03 AM
Unknown Object (File)
Aug 15 2023, 5:40 AM
Unknown Object (File)
Jun 26 2023, 11:32 AM
Unknown Object (File)
Jun 26 2023, 5:21 AM
Subscribers

Details

Reviewers
mat
Summary
  • Removal of shar instructions
    • We are at the state of shar/diff, while diff is preferred, as it makes it easier for developers/commiters to use
    • Been encouraging the use of diffs over shar, in which this is the next step

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 19170
Build 18790: arc lint + arc unit

Event Timeline

I disagree with this change. We support submitting new ports using shar, so it needs to be documented.

In D16896#360150, @mat wrote:

I disagree with this change. We support submitting new ports using shar, so it needs to be documented.

I can see both sides :) though, when trying to review the patch, when it is a shar file, in bugzilla; can't just look at them. Have to download then use an editor to look at it. Also, could be considered a possible "security" issue. If someone new to UNIX-like systems was running as root and didn't checkout what the shar file contained; they could execute it and it cause malicious activity.

linimon retitled this revision from Remove shar file instructions to Porters Handbook: Remove shar file instructions.Aug 27 2018, 9:42 AM

We have Tools/scripts/sunshar.rb to safely unpack shar archives.

Also, doing everything as root is a bad idea. The only thing you need to run as root when testing ports is poudriere testport. Everything else can, and should, be done as a regular user.