base: improve livecheck.type moddate (review request)

Bradley Giesbrecht pixilla at macports.org
Fri Sep 12 01:16:01 PDT 2014


On Sep 12, 2014, at 1:03 AM, Joshua Root <jmr at macports.org> wrote:

> 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.

I want to replace isnewer with getmoddate.


Regards,
Bradley Giesbrecht (pixilla)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 455 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <https://lists.macosforge.org/pipermail/macports-dev/attachments/20140912/70c4cc36/attachment-0001.sig>


More information about the macports-dev mailing list