Re: [gentoo-portage-dev] dodoc creating a symlink to distfiles?

2011-04-11 Thread Zac Medico 
On 04/11/2011 04:32 PM, Zac Medico wrote:
> On 04/11/2011 10:14 AM, Zac Medico wrote:
>> On 04/11/2011 09:04 AM, James Cloos wrote:
>>> the recently added sci-math/minisat ebuild has this:
>>>
>>>dodoc "${DISTDIR}"/MiniSat.pdf || die
>>>
>>> which results in:
>>>
>>> sym /usr/share/doc/minisat-2.2.0-r2/MiniSat.pdf -> 
>>> /usr/portage/distfiles/MiniSat.pdf 1302537422
>>>
>>> Is that a portage issue or an issue with the ebuild?
>>>
>>> My portage is a few days old; I last merged commit 1d6e6b2fe3b01 from 27 
>>> March.
>>>
>>> -JimC
>>
>> It's a side-effect from this fix which makes dodoc preserve symlinks
>> like doins does in EAPI 4:
>>
>>   https://bugs.gentoo.org/show_bug.cgi?id=356461
>>
>> I guess that we could make doins automatically dereference any symlinks
>> that refer to files absolute paths. For symlinks that refer to
>> directories by absolute paths, we could continue to preserve the symlink
>> as we do now.
> 
> Here's the patch:
> 
> http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=2e334d77e3d1836ab6ba5dfc1700e90f9599d4d3

Now it's constrained to $PORTAGE_ACTUAL_DISTDIR/ so that we don't stray
from PMS:

http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=de9a536f470919651e83ece51923594e8605781b
-- 
Thanks,
Zac

Re: [gentoo-portage-dev] dodoc creating a symlink to distfiles?

2011-04-11 Thread Zac Medico 
On 04/11/2011 10:14 AM, Zac Medico wrote:
> On 04/11/2011 09:04 AM, James Cloos wrote:
>> the recently added sci-math/minisat ebuild has this:
>>
>>dodoc "${DISTDIR}"/MiniSat.pdf || die
>>
>> which results in:
>>
>> sym /usr/share/doc/minisat-2.2.0-r2/MiniSat.pdf -> 
>> /usr/portage/distfiles/MiniSat.pdf 1302537422
>>
>> Is that a portage issue or an issue with the ebuild?
>>
>> My portage is a few days old; I last merged commit 1d6e6b2fe3b01 from 27 
>> March.
>>
>> -JimC
> 
> It's a side-effect from this fix which makes dodoc preserve symlinks
> like doins does in EAPI 4:
> 
>   https://bugs.gentoo.org/show_bug.cgi?id=356461
> 
> I guess that we could make doins automatically dereference any symlinks
> that refer to files absolute paths. For symlinks that refer to
> directories by absolute paths, we could continue to preserve the symlink
> as we do now.

Here's the patch:

http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=2e334d77e3d1836ab6ba5dfc1700e90f9599d4d3
-- 
Thanks,
Zac

Re: [gentoo-portage-dev] dodoc creating a symlink to distfiles?

2011-04-11 Thread Zac Medico 
On 04/11/2011 09:04 AM, James Cloos wrote:
> the recently added sci-math/minisat ebuild has this:
> 
>dodoc "${DISTDIR}"/MiniSat.pdf || die
> 
> which results in:
> 
> sym /usr/share/doc/minisat-2.2.0-r2/MiniSat.pdf -> 
> /usr/portage/distfiles/MiniSat.pdf 1302537422
> 
> Is that a portage issue or an issue with the ebuild?
> 
> My portage is a few days old; I last merged commit 1d6e6b2fe3b01 from 27 
> March.
> 
> -JimC

It's a side-effect from this fix which makes dodoc preserve symlinks
like doins does in EAPI 4:

  https://bugs.gentoo.org/show_bug.cgi?id=356461

I guess that we could make doins automatically dereference any symlinks
that refer to files absolute paths. For symlinks that refer to
directories by absolute paths, we could continue to preserve the symlink
as we do now.
-- 
Thanks,
Zac

[gentoo-portage-dev] dodoc creating a symlink to distfiles?

2011-04-11 Thread James Cloos 
the recently added sci-math/minisat ebuild has this:

   dodoc "${DISTDIR}"/MiniSat.pdf || die

which results in:

sym /usr/share/doc/minisat-2.2.0-r2/MiniSat.pdf -> 
/usr/portage/distfiles/MiniSat.pdf 1302537422

Is that a portage issue or an issue with the ebuild?

My portage is a few days old; I last merged commit 1d6e6b2fe3b01 from 27 March.

-JimC
-- 
James Cloos  OpenPGP: 1024D/ED7DAEA6