[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