[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 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.

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