[MacPorts] #44117: rsync-3.1.1 : Diffs to bring port up to 3.1.1
#44117: rsync-3.1.1 : Diffs to bring port up to 3.1.1 ----------------------+-------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.0 Keywords: rsync | Port: rsync ----------------------+-------------------------------- -- Ticket URL: <https://trac.macports.org/ticket/44117> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync-3.1.1 : Diffs to bring port up to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Changes (by mf2k@…): * keywords: rsync => haspatch maintainer * version: 2.3.0 => -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by ryandesign@…): I'm curious why you've added this line: {{{ configure.cflags "-Os -I${prefix}/include" }}} and also why this line already exists in the portfile: {{{ configure.ldflags "-L${prefix}/lib" }}} since both of those values are the defaults. -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Changes (by ryandesign@…): * cc: mojca@… (added) Comment: You might also take a look at mojca's comments about 3.1.1pre2 in comment:ticket:40803:9 and see if any of them still apply to 3.1.1. -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by ryandesign@…): Replying to [comment:3 ryandesign@…]:
and also why this line already exists in the portfile:
{{{ configure.cflags "-Os -I${prefix}/include" }}}
since both of those values are the defaults.
Hmm, actually the `-I` flag would be in cppflags by default, not cflags. Are we saying rsync's build system erroneously does not understand cppflags? -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by mojca@…): If you take a look at my patch, I actually had to remove the `cflags` to make the compilation work in the first place. But I submitted a bug report, so maybe it works with that flag again (which is not to say that the flag is needed). My removal of patches in the other ticket was wrong (sorry for the mistake, but it's weird to have a bunch of official patches and I still find the concept a bit weird). The patches could actually be fetched from http://rsync.samba.org/ftp/rsync/src/rsync-patches-3.1.1.tar.gz automatically as opposed to making copies of those files locally and keep updating them. What do you think? So I would suggest to: * remove those explicit flag unless you can explain why they are needed * try to fetch both compresed files and apply patches from there -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:6> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by jimjag@…): The cflags line is not new but the ldflags line is. The latter is due to changes from 3.1.0+ -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:7> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by jimjag@…): Regarding the patches: Please note that 2 of those files are not included in the patches tarball. I think it's best to continue as is with as few changes as needed, -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:8> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by mojca@…): But why is `ldflags` needed? -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:9> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by jimjag@…): Confirmed that rsync-3.1.1 works as expected and passes bbouncer tests w/o the configure.cflags and configure.ldflags lines. Will commit (and release) as soon as I get commit privs as rsync maintainer. -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:10> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by mojca@…): Replying to [comment:10 jimjag@…]:
Will commit (and release) as soon as I get commit privs as rsync
maintainer. Did you recently get an answer that you are getting commit rights? -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:11> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by jimjag@…): Considering that I've been maintaining rsync for ~2years, I assume that my request for commit will be approved. -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:12> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by mojca@…): But I wouldn't count on reply being fast enough, they have a long backlog. I know that some people waited for months to get the answer (not for the actual commit rights, but just to get the decision). That's why I asked. It might be better to commit your changes now to get the updated port than to wait indefinitely for the commit rights. I can commit the changes, but just in case I would like to see the patch from you to make sure that I don't misunderstand and commit the wrong thing. -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:13> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: Resolution: | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by jimjag@…): The patch is to be attached... awaiting someone w/ commit to apply... -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:14> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: closed Priority: Normal | Milestone: Component: ports | Version: Resolution: fixed | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Changes (by mojca@…): * status: new => closed * resolution: => fixed Comment: Thank you. r121462. -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:15> MacPorts <http://www.macports.org/> Ports system for OS X
#44117: rsync: update to 3.1.1 -----------------------+--------------------------------- Reporter: jimjag@… | Owner: macports-tickets@… Type: update | Status: closed Priority: Normal | Milestone: Component: ports | Version: Resolution: fixed | Keywords: haspatch maintainer Port: rsync | -----------------------+--------------------------------- Comment (by jimjag@…): Thx! -- Ticket URL: <https://trac.macports.org/ticket/44117#comment:16> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts