[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [tyndur-devel] [PATCH 2/4] build: Temporäre Dateien gehören nach /tmp



On 10.10.2012 23:25, Kevin Wolf wrote:
> On Wed, Oct 10, 2012 at 10:40:58AM +0200, MNemo wrote:
>> On 09.10.2012 18:37, Kevin Wolf wrote:
>>> On Wed, Oct 03, 2012 at 04:55:22PM +0200, Andreas Freimuth wrote:
>>>>  * build/scripts/image_floppy_grub1: Die Konfigurationsdatei für MTOOLS
>>>>    wird nur kurz benötigt und dann gleich wieder gelöscht. Also ab damit
>>>>    nach /tmp
>>>>
>>>> Signed-off-by: Andreas Freimuth <m.nemo@xxxxxxx>
>>>> ---
>>>>  build/scripts/image_floppy_grub1 | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/build/scripts/image_floppy_grub1 b/build/scripts/image_floppy_grub1
>>>> index 463b117..f4f81a3 100755
>>>> --- a/build/scripts/image_floppy_grub1
>>>> +++ b/build/scripts/image_floppy_grub1
>>>> @@ -45,7 +45,7 @@ export LOST_BUILDMK_ROOT="`pwd`"
>>>>  source config.sh
>>>>  
>>>>  
>>>> -export MTOOLSRC=lostmtools.conf
>>>> +export MTOOLSRC=/tmp/lostmtools.conf$$
>>>
>>> Wäre nicht sowas wie ${TMPDIR:-/tmp} besser? Nicht, dass es irgendjemand
>>> auf einem System benutzen würde, auf dem das anders ist (oder
>>> funktioniert dieses Skript unter Windows?), aber trotzdem...
>> Ja… das sieht Sinnvoll aus. Ich habe glaube ich auch zu erst mit TMPDIR
>> probiert aber das ist bei mir nicht gesetzt, und den Syntax
>> ${var:-default} kannte ich noch nicht.
> 
> Hab das mal eingebaut und gepusht. Ich glaube, damit müssten jetzt alle
> deine Patches drin sein. Du kannst ja mal rebasen, dass du siehst, ob
> noch was fehlt.
Jo. Hast alles erwischt. Danke.

> 
> Kevin
> _______________________________________________
> tyndur-devel mailing list
> tyndur-devel@xxxxxxxxxx
> http://list.tyndur.org/mailman/listinfo/tyndur-devel
>