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

[Kevin Wolf <kevin@xxxxxxxxxx>]

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.

Kevin