[MacPorts] #47231: resolve minor nits with par
MacPorts
noreply at macports.org
Fri Apr 3 11:10:22 PDT 2015
#47231: resolve minor nits with par
--------------------------+-----------------------
Reporter: lists@… | Owner: qbarnes@…
Type: enhancement | Status: new
Priority: Low | Milestone:
Component: ports | Version: 2.3.3
Resolution: | Keywords: haspatch
Port: par |
--------------------------+-----------------------
Comment (by ionic@…):
Replying to [comment:7 qbarnes@…]:
> Ah, ok. Nothing technically wrong with the way it was other than
violating convention.
Yep, it's merely an enhancement.
> It is a change to bring it into alignment with other packages.
The vast majority of packages either don't specify the mode or use `755`.
There are some ports currently installing with `555` for binaries, but
that's what these tickets are for (or somesuch.)
> In that case, sounds good then, especially with removing the -m option
letting xinstall default rather than rehardcoding a new value.
>
> So not bumping the rev is simply to save triggering a rebuild for people
for a inconsequential update?
Well, it's for saving triggering a rebuild for people/users... mind you,
that's "wrong" to my mind, as the filesystem and `Portfile` will end up
diverging, even though this diversion is rather cosmetic and not
influencing how a package works. I, personally, still prefer consistency
over the comfort of not having to rebuild packages. Most other MacPorts
developers seem to think otherwise, however, so I don't really care.
> If so, the patch looks good to me and should be applied. What's the
next step? I don't have commit access.
I can apply everything, as long as the maintainer gives explicit
permission.
--
Ticket URL: <https://trac.macports.org/ticket/47231#comment:8>
MacPorts <https://www.macports.org/>
Ports system for OS X
More information about the macports-tickets
mailing list