Skip to content

Commit

Permalink
do not run Init.sh in terminal, GRASS is already started if QGIS is r…
Browse files Browse the repository at this point in the history
…un from GRASS shell or everything is set by QgsGrass::openMapset

git-svn-id: http://svn.osgeo.org/qgis/trunk@12995 c8812cc2-4d05-0410-92ff-de0c093fc19c
  • Loading branch information
rblazek committed Mar 3, 2010
1 parent 3ae8532 commit 9f48910
Showing 1 changed file with 16 additions and 30 deletions.
46 changes: 16 additions & 30 deletions src/plugins/grass/qgsgrassshell.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -46,38 +46,18 @@ QgsGrassShell::QgsGrassShell( QgsGrassTools *tools, QTabWidget *parent, const ch
connect( copyShortcut, SIGNAL( activated() ), mTerminal, SLOT( copyClipboard() ) );

// TODO: find a better way to manage the lockfile.
// Locking should not be done here, a mapset is either locked by GRASS if QGIS is started from GRASS or it is created by QgsGrass::openMapset
/*
mLockFilename = QgsGrass::lockFilePath();
QFile::remove( mLockFilename + ".qgis" );
if ( !QFile::rename( mLockFilename, mLockFilename + ".qgis" ) )
{
QMessageBox::warning( this, tr( "Warning" ), tr( "Cannot rename the lock file %1" ).arg( mLockFilename ) );
}
*/

mTerminal->setSize( 80, 25 );
mTerminal->startShellProgram();
// using -text option GRASS_GUI is owerridden so we have to reset it
QgsDebugMsg ( "gisrc = " + QgsGrass::gisrcFilePath() );
QFile in( QgsGrass::gisrcFilePath() );
if ( in.open( QIODevice::ReadOnly ) )
{
char buf[1000];
QString oldGui;
while ( in.readLine( buf, 1000 ) != -1 )
{
QString line = buf;
QStringList kv = line.split ( ':' );
if ( kv.length() == 2 && kv.at(0) == "GRASS_GUI" )
{
oldGui = kv.at(1).trimmed ();
break;
}
}
if ( ! oldGui.isEmpty() )
{
QString cmd = QString("g.gisenv set=GRASS_GUI=") + oldGui + "\n";
mTerminal->sendText( cmd );
}
}
mTerminal->setFocus( Qt::MouseFocusReason );
}

Expand All @@ -91,11 +71,13 @@ void QgsGrassShell::closeShell()
mTabWidget->removeTab( index );

// TODO: find a better way to manage the lockfile.
// No locking should be done here, see above
/*
if ( !QFile::rename( mLockFilename + ".qgis", mLockFilename ) )
{
QMessageBox::warning( this, tr( "Warning" ), tr( "Cannot rename the lock file %1" ).arg( mLockFilename ) );
}

*/
this->deleteLater();
}

Expand All @@ -104,18 +86,22 @@ void QgsGrassShell::initTerminal( QTermWidget *terminal )
QStringList env( "" );
QStringList args( "" );

QString shellProgram = QString( "%1/etc/Init.sh" ).arg( ::getenv( "GISBASE" ) );
// GRASS Init.sh should not be started here, it is either run when GRASS is started if QGIS is run from GRASS shell or everything (set environment variables and lock mapset) is done in QgsGrass::openMapset
//QString shellProgram = QString( "%1/etc/Init.sh" ).arg( ::getenv( "GISBASE" ) );

terminal->setShellProgram( shellProgram );
//terminal->setShellProgram( shellProgram );
env << "TERM=vt100";
env << "GISRC_MODE_MEMORY";
// This is also overriden by Init.sh, it should not be run at all, either QGIS is started from shell or a mapset is open from QGIS, Init.sh opens the session second time
// TODO: we should check if these environment variable were set by user before QGIS was started
env << "GRASS_HTML_BROWSER=" + QgsApplication::pkgDataPath() + "/grass/bin/qgis.g.browser";
env << "GRASS_WISH=wish";
env << "GRASS_TCLSH=tclsh";
env << "GRASS_PYTHON=python";

args << "-text";
args << QString( "%1/%2/%3" ).arg( QgsGrass::getDefaultGisdbase() ).arg( QgsGrass::getDefaultLocation() ).arg( QgsGrass::getDefaultMapset() );
//args << "-text";
//args << QString( "%1/%2/%3" ).arg( QgsGrass::getDefaultGisdbase() ).arg( QgsGrass::getDefaultLocation() ).arg( QgsGrass::getDefaultMapset() );

terminal->setArgs( args );
//terminal->setArgs( args );
terminal->setEnvironment( env );

// Look & Feel
Expand Down

0 comments on commit 9f48910

Please sign in to comment.