Coreutils update/mktemp

[Kok, Auke]

Zbigniew Luszpinski wrote:
> Tuesday 22 of April 2008 00:18:47 Zbigniew Luszpinski wrote:
>> Monday 21 of April 2008 18:34:50 Kok, Auke wrote:
>>> Zbigniew Luszpinski wrote:
>>>> * removed sedits added in ver 6.10
>>>> * added symlinks to /usr/bin/{uname|arch}
>>>> * now conflicting shadow procps mktemp net-tools
>>> let's not do this... this can still break things.. please check my other
>>> mail I just sent to the list a second ago.
>> What about:
>> /usr/share/man/man1/mktemp.1.gz
>> How to resolve this conflict?
>>
>> How about such fix:
>> module: mktemp, change install prefix to /bin:
>> to install mktemp to /bin/mktemp
>> modify Makefile to not install:
>> /usr/share/man/man1/mktemp.1.gz
>> (and commit this update long before coreutils commit to make sure it will
>> be updated before coreutils will lin it in pre_build).
>>
>> then
>> module: coreutils
>> add to PRE_BUILD:
>> lin -c mktemp
>> to install updated mktemp module to /bin before lining coreutils.
>>
>> Thanks to this we have advantages:
>> *mktemp will move itself to /bin and will not install mktemp.1.gz,
>> then coreutils will install files to /usr/bin and /usr/share/man.
>> *when mktemp module is going to be removed the default /usr/bin
>> and /usr/share/man files installed by coreutuils will be untouched by lrm
>> mktemp.
>> *thanks to lin -c mktemp in pre_build we know for sure current, working
>> mktemp is indeed in /bin. Then we again know new mktemp will go to
>> /usr/bin. (this is protection against 'no mktemp' problem when something
>> goes wrong or an user has non-default file/directory layout). The 'no
>> mktemp' problem is very painful as lin does not work without mktemp app.
> 
> Proof of concept commited to Zbiggy's workshop/zbiggy-WIP branch on
> Tue, 22 Apr 2008 18:45:12 +0000 (20:45 +0200)
> 
> The result:
> lvu install mktemp | grep mktemp
> /bin/mktemp
> 
> lvu install coreutils | grep mktemp
> /usr/bin/mktemp
> /usr/share/man/man1/mktemp.1.gz


perfect... that's looks great. I looked into your branch and the commit you made
does the right thing - please merge this in to master!

thanks zbiggy!

Auke