base: improve livecheck.type moddate (review request)

Joshua Root jmr at macports.org
Fri Sep 12 01:03:14 PDT 2014


On 2014-9-12 14:31 , Bradley Giesbrecht wrote:
> MacPorts Devs,
> 
> 
> I am seeking comments on the patches [1][2] to port livecheck in ticket 44894 [3].
> 
> In curl.c, the getmoddate function is copied from the isnewer function with minor changes to return the remote file moddate.
> 
> In portfilecheck.tcl, I have add a "livecheck.type moddate2" for testing. I have attempted to have moddate2 default to the current behavior or moddate.
> 
> The goal is to improve "livecheck.type moddate" by making the comparison date settable in the Portfile as opposed to the current behavior of only using the Portfile mtime for comparison.
> 
> The ticket 44894 has example usage in comment 1.
> 
> 
> [1] https://trac.macports.org/attachment/ticket/44894/patch-base-src-port1.0-portlivecheck.tcl.diff
> [2] https://trac.macports.org/attachment/ticket/44894/patch-base-src-pextlib1.0-curl.c.diff
> [3] https://trac.macports.org/ticket/44894#comment:1

Please don't just add another copy of mostly identical code. Instead,
modify isnewer to use getmoddate.

- Josh


More information about the macports-dev mailing list