[114687] trunk/dports/_resources/port1.0/group/github-1.0.tcl
Sean Farley
sean at macports.org
Fri Dec 13 16:31:49 PST 2013
ryandesign at macports.org writes:
> On Dec 13, 2013, at 16:49, sean at macports.org wrote:
>
>> Revision
>> 114687
>> Author
>> sean at macports.org
>> Date
>> 2013-12-13 14:49:52 -0800 (Fri, 13 Dec 2013)
>> Log Message
>>
>> github-1.0: use worksrpath variable when renaming folder; fixes #41797
>> Modified Paths
>>
>> • trunk/dports/_resources/port1.0/group/github-1.0.tcl
>> Diff
>>
>> Modified: trunk/dports/_resources/port1.0/group/github-1.0.tcl (114686 => 114687)
>>
>> --- trunk/dports/_resources/port1.0/group/github-1.0.tcl 2013-12-13 22:30:27 UTC (rev 114686)
>> +++ trunk/dports/_resources/port1.0/group/github-1.0.tcl 2013-12-13 22:49:52 UTC (rev 114687)
>>
>> @@ -87,7 +87,7 @@
>>
>> [llength [glob -nocomplain ${workpath}/*]] > 0} {
>>
>> if {[file exists [glob ${workpath}/${github.author}-${github.project}-*]] && \
>>
>> [file isdirectory [glob ${workpath}/${github.author}-${github.project}-*]]} {
>>
>> - move [glob ${workpath}/${github.author}-${github.project}-*] ${workpath}/${name}-${version}
>>
>> + move [glob ${workpath}/${github.author}-${github.project}-*] ${worksrcpath}
>
> I had mentioned on the mailing list that it would be nice to *not* use worksrcpath here, so that we could finally fix the bug which prevented ports using the github portgroup from being able to use the worksrcpath variable effectively.
>
> https://lists.macosforge.org/pipermail/macports-dev/2013-December/025456.html
>
> I had suggested using “${distname}” instead. Markus went with "${workpath}/${name}-${version}”; I’m not sure why.
Ok, I must have missed that. It seems that this needs some refactoring /
abstraction so that another port group can change worksrcpath (or
distname) and not affect ports that use worksrcpath in their portfile. I
don't really care how it's solved as long as it doesn't break
compatibility.
More information about the macports-dev
mailing list