Re: [129459] trunk/dports/multimedia/mpv

2014-12-14 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 14.12.2014 05:40 AM, Ryan Schmidt wrote: On Dec 13, 2014, at 10:01 PM, Mihai Moldovan wrote: On 14.12.2014 04:51 AM, Ryan Schmidt wrote: If it works for your use case in mpv, and still works for other ports using the github portgroup, go

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Ryan Schmidt
On Dec 13, 2014, at 7:49 AM, io...@macports.org wrote: Revision 129459 Author io...@macports.org Date 2014-12-13 05:49:48 -0800 (Sat, 13 Dec 2014) Log Message mpv: update to 0.7.1. Apply 10.6 patches cleanly against changes base. --- trunk/dports/multimedia/mpv/Portfile

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 13.12.2014 08:16 PM, Ryan Schmidt wrote: +# Current waf doesn't support --nocache anymore. Set by PortGroup. +configure.args-delete --nocache Sounds like the waf 1.0 portgroup should be modified then. I couldn't find a port called

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Ryan Schmidt
On Dec 13, 2014, at 3:29 PM, Mihai Moldovan wrote: On 13.12.2014 08:16 PM, Ryan Schmidt wrote: +# Current waf doesn't support --nocache anymore. Set by PortGroup. +configure.args-delete --nocache Sounds like the waf 1.0 portgroup should be modified then. I couldn't find a port

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 14.12.2014 01:04 AM, Ryan Schmidt wrote: On Dec 13, 2014, at 3:29 PM, Mihai Moldovan wrote: On 13.12.2014 08:16 PM, Ryan Schmidt wrote: +# Current waf doesn't support --nocache anymore. Set by PortGroup. +configure.args-delete --nocache

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 [re-sent with correct mail address, so that the post arrives at mp-dev] On 14.12.2014 03:14 AM, Mihai Moldovan wrote: On 14.12.2014 01:04 AM, Ryan Schmidt wrote: On Dec 13, 2014, at 3:29 PM, Mihai Moldovan wrote: On 13.12.2014 08:16 PM, Ryan

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Ryan Schmidt
On Dec 13, 2014, at 8:26 PM, Mihai Moldovan wrote: So, as soon as I use master_sites-append http://ftp.waf.io/pub/release:waf, this stuff breaks. How is this even possible? This stuff being: - worksrcdir is correctly set to

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 14.12.2014 04:11 AM, Ryan Schmidt wrote: On Dec 13, 2014, at 8:26 PM, Mihai Moldovan wrote: So, as soon as I use master_sites-append http://ftp.waf.io/pub/release:waf, this stuff breaks. How is this even possible? Not base; the github

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Ryan Schmidt
On Dec 13, 2014, at 9:25 PM, Mihai Moldovan wrote: On 14.12.2014 04:11 AM, Ryan Schmidt wrote: On Dec 13, 2014, at 8:26 PM, Mihai Moldovan wrote: So, as soon as I use master_sites-append http://ftp.waf.io/pub/release:waf, this stuff breaks. How is this even possible? Not base; the

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 14.12.2014 04:34 AM, Ryan Schmidt wrote: Those checks went through many revisions, not all by me, to ensure that we only rename the directory when it is correct to do so. I agree the complete set of checks seems overly cautious right now, I

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 14.12.2014 04:38 AM, Mihai Moldovan wrote: On 14.12.2014 04:34 AM, Ryan Schmidt wrote: Those checks went through many revisions, not all by me, to ensure that we only rename the directory when it is correct to do so. I agree the complete set

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Ryan Schmidt
On Dec 13, 2014, at 9:43 PM, Mihai Moldovan wrote: Ok, proposing this. Actually, more like this. [using lsearch on master_sites] That might do it! ___ macports-dev mailing list macports-dev@lists.macosforge.org

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 14.12.2014 04:45 AM, Ryan Schmidt wrote: On Dec 13, 2014, at 9:43 PM, Mihai Moldovan wrote: Ok, proposing this. Actually, more like this. [using lsearch on master_sites] That might do it! This checks if github.master_sites is

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Ryan Schmidt
On Dec 13, 2014, at 9:49 PM, Mihai Moldovan wrote: This checks if github.master_sites is included somewhere in master_sites. If you'd prefer github.master_sites to be the first entry in master_sites, the check should trivially be changed to == 0. But I think checking that github.master_sites

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Mihai Moldovan
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 14.12.2014 04:51 AM, Ryan Schmidt wrote: If it works for your use case in mpv, and still works for other ports using the github portgroup, go ahead. Hum, there are 468 ports making use of the github PortGroup. Testing all of them is out of

Re: [129459] trunk/dports/multimedia/mpv

2014-12-13 Thread Ryan Schmidt
On Dec 13, 2014, at 10:01 PM, Mihai Moldovan wrote: On 14.12.2014 04:51 AM, Ryan Schmidt wrote: If it works for your use case in mpv, and still works for other ports using the github portgroup, go ahead. Hum, there are 468 ports making use of the github PortGroup. Testing all of them is