hardcoded /dev/null

[hamish]

Hi,

imagery/i.cluster/main.c
raster/r.out.mpeg/main.c
raster/r.topmodel/misc.c
lib/gis/opencell.c  G__open_cell_old()

all hardcode to /dev/null which doesn't exist on Windows.

perhaps

#ifdef __MINGW32__
#define NULLDEV "NUL:"
#else
#define NULLDEV "/dev/null"
#endif

or somesuch? (no idea)

d.ask and XDRIVER do too, but they don't work on MS Windows.

Hamish

[glynn]

Replying to hamish:

perhaps

#ifdef __MINGW32__
#define NULLDEV "NUL:"
#else
#define NULLDEV "/dev/null"
#endif

or somesuch? (no idea)

See G_DEV_NULL in gis.h

[hamish]

i.cluster, r.topmodel, and lib/gis/opencell.c updated in r36143 - r36148.

For grass 6 I am not really sure what to do with r.mapcalc from r.topmodel/misc.c. I can do

  G_putenv("GRASS_VERBOSE", "0");

but then how to undo that as soon as r.mapcalc is done? ie so you don't override user's verbosity setting for everything that happens afterwards in the module. on unix you could put "GRASS_VERBOSE=0 r.mapcalc ..." on the same line, dunno if that works for windows.

n.b. old >/dev/null didn't catch G_percent() as it is to stderr not stdout.

Also I am not sure how to fix r.out.mpeg (sends 2> /dev/null), so that one remains to be fixed.

Hamish

[neteler]

The remaining /dev/null are

find . -name '*.c' | xargs grep '/dev/null'
./raster/r.out.mpeg/main.c:    if (256 == G_system("ppmtompeg 2> /dev/null"))
./raster/r.out.mpeg/main.c:    else if (256 == G_system("mpeg_encode 2> /dev/null"))
./raster/r.out.mpeg/main.c:     sprintf(cmd, "%s %s 2> /dev/null > /dev/null", encoder, mpfilename);
./raster/r.out.mpeg/main.c:     sprintf(cmd, "%s %s 2> /dev/null > /dev/null", encoder, mpfilename);
./raster/r.out.mpeg/main.c:         sprintf(cmd, "cd %s; \\ls %s >> %s 2> /dev/null",
./display/drivers/XDRIVER/Serve_Xevent.c:    open("/dev/null", O_RDONLY);
./display/drivers/XDRIVER/Serve_Xevent.c:    open("/dev/null", O_WRONLY);
./display/drivers/XDRIVER/Serve_Xevent.c:    open("/dev/null", O_WRONLY);
./display/d.ask/main.c: fd = popen("d.menu tcolor=red > /dev/null", "w");
./visualization/xganim/main.c:      sprintf(cmd, "cd %s; \\ls %s >> %s 2> /dev/null", path, wildarg,

[hamish]

as in comment:2 I'm not sure how to fix r.out.mpeg. XDRIVER and d.ask aren't an issue for WinGrass. I'm not sure if xganim could be made to work on WinGrass or not, but it's better replaced by the wxGUI animator version. (?)

So I'd be happy to close the bug as long as r.out.mpeg could get fixed.

Hamish

[glynn]

Replying to hamish:

as in comment:2 I'm not sure how to fix r.out.mpeg.

G_system("ppmtompeg 2>" G_DEV_NULL);

A more significant issue is gee_wildfiles(), which does:

    sprintf(cmd, "cd %s; \\ls %s >> %s 2> /dev/null",
	    path, wildarg, tfile);

If you want a complete replacement, the 7.0 version of xganim uses G_spawn_ex() and g.mlist.

I'm not sure if xganim could be made to work on WinGrass or not, but it's better replaced by the wxGUI animator version. (?)

The 6.x xganim requires Motif and an X server, which is probably more trouble than it's worth. Also, on Windows and MacOSX you would need to either use Lesstif (which is still "beta" quality and barely maintained) or buy a commercial version of Motif; the OpenMotif licence is limited to open-source platforms.

[hamish]

r.out.mpeg still needs fixing.

[wenzeslaus]

Replying to glynn:

Replying to hamish:

I'm not sure if xganim could be made to work on WinGrass or not, but it's better replaced by the wxGUI animator version. (?)

The 6.x xganim requires Motif and an X server, which is probably more trouble than it's worth. Also, on Windows and MacOSX you would need to either use Lesstif (which is still "beta" quality and barely maintained) or buy a commercial version of Motif; the OpenMotif licence is limited to open-source platforms.

Just for the record, xganim was removed from GRASS 7 in favor of g.gui.animation.

For trunk, both egrep -Irn "NULLDEV" and egrep -Irn "/dev/null" return nothing for raster/r.out.mpeg.