Opened 15 years ago

Closed 14 years ago

#1799 closed bug (fixed)

g.manual not working

Reported by: pcav Owned by: rugginoso
Priority: minor: annoyance Milestone: Version 1.5.0
Component: GRASS Version: Trunk
Keywords: Cc: neteler
Must Fix for Release: No Platform: All
Platform Version: Awaiting user input: no

Description

The module g.manual does not work, because it misses the x-www-browser parameter:

g.manual -i

Starting browser <> for module index...

/usr/lib/grass64/scripts/g.manual: line 79: : command not found

From the grass shell there is no problem:

Starting browser <x-www-browser> for module index...

Change History (10)

comment:1 by lutra, 15 years ago

Under Ubuntu 9.04 I get

g.manual -i



Starting browser <> for module index...

/usr/local/grass-6.4.0RC4/scripts/g.manual: 122: : Permission denied


Successfully finished

comment:2 by pcav, 14 years ago

In r12197 is slightly different:

g.manual -i

Starting browser for module index...

/usr/lib/grass64/scripts/g.manual: line 79: /usr/share/qgis/grass/bin/qgis.g.browser: No such file or directory

comment:3 by lutra, 14 years ago

it works under Ubuntu now, using r12917 and GRASS installed from the ubuntugis ppa repository.

comment:4 by lutra, 14 years ago

Platform: DebianAll

it does not under windows (xp)

C:\>g.manual -i
Starting browser <C:\Program Files\Internet Explorer\iexplore.exe> for
module index...
C:\OSGeo4W\apps\grass\GRASS-~1.0SV/scripts/g.manual: C:\Program Files\Internet E
xplorer\iexplore.exe: command not found

comment:5 by rblazek, 14 years ago

It is bug in g.manual, I guess the problem is spaces in explorer path.

See also http://trac.osgeo.org/grass/ticket/980 and http://trac.osgeo.org/grass/ticket/655

To avoid problems with default browser we have qgis.g.browser in qgis which is using QDesktopServices::openUrl but GRASS_HTML_BROWSER was not set to qgis.g.browser for windows shell. It set from r:13002.

Unfortunately g.manual -i does not work even with qgis.g.browser, because QDesktopServices::openUrl (windows) fails to open 'file://c:...' it must be 'file:///c:...' (3 slashes) so I changed it so that ti add one slash to path on win.

To be check with next build.

comment:6 by rblazek, 14 years ago

Still does not work because the drive letter seems to be somehow lost in QUrl parser for 'file://c:...' and late setPath does not help. Another attempt commited which modifies the string before it is passed to setUrl.

in reply to:  5 comment:7 by neteler, 14 years ago

Cc: neteler added

Replying to rblazek:

It is bug in g.manual, I guess the problem is spaces in explorer path.

See also http://trac.osgeo.org/grass/ticket/980

this has been fixed 13/mar/2010 in grass6.4.0svn, grass6.5.svn, grass7.svn.

and http://trac.osgeo.org/grass/ticket/655

fixed in GRASS 07/oct/09.

Perhaps this bug is now solved?

Markus

comment:8 by rblazek, 14 years ago

Resolution: fixed
Status: newclosed

Uff, it works but it was necessary to prepend "/" to the path BEFORE it is passed to QUrl. QUrl is doing some tricky parsing/manipulation. I won't be surprised if next Qt release breaks it again.

comment:9 by pcav, 14 years ago

Resolution: fixed
Status: closedreopened

Seems broken once more:

g.manual -i

Starting browser for module index... /usr/lib/grass64/scripts/g.manual: line 79: /usr/share/qgis/grass/bin/qgis.g.browser: No such file or directory Successfully finished

comment:10 by jef, 14 years ago

Resolution: fixed
Status: reopenedclosed

I guess you use the debian package, which was lacking qgis.g.browser

Note: See TracTickets for help on using tickets.