Opened 8 years ago

Closed 8 years ago

#771 closed defect (fixed)

AtlasStyler does not open from ISO, but seems to work from VM

Reported by: camerons Owned by: live-demo@…
Priority: major Milestone:
Component: OSGeoLive Keywords: 5.0, atlasstyler, iso
Cc:

Description

On 23/08/11 00:35, Angelos Tzotsos wrote:

Hi all,

I have been testing 5.0RC1 for a couple of days. I use a netbook for my tests with 1GB RAM and I use the iso version (loaded into a usb).

The main problems I have seen so far are:

...

  1. atlasstyler - does not open at all from the launcher (Hamish commented that his copy works ok, perhaps it is an ISO thing...)

Change history (15)

comment:1 Changed 8 years ago by alfonx

Cc: alfonx added

I am trying to reproduce it with ISO within the next 48h. Sorry, stupid question, but I download the iso here: http://download.osgeo.org/livedvd/release/5.0/osgeo-live-5.0.iso ? which seems to be 5.0.RC2 at the moment.

comment:2 Changed 8 years ago by camerons

You can download the latest image (currently at osgeo-live 5.0rc2) at http://live.osgeo.org/dev/build

comment:3 Changed 8 years ago by hamish

works for me in 5.0rc2 (in a VM).

just takes a while to start up.

shrug, Hamish

comment:4 Changed 8 years ago by hamish

ps- if it fails from the menu, how does it do from the command line?

comment:5 Changed 8 years ago by alfonx

Reproducable on RC2 ISO: My coworker tells me, that

/usr/share/atlasstyler/start_AtlasStyler.sh

doesn't have the executable flag set?! This file is extracted from/ created by the atlasstyler.deb package WITH executable flag set?! How could it be, that the flag is getting lost between VM and ISO?

comment:6 in reply to:  5 Changed 8 years ago by hamish

Replying to alfonx:

Reproducable on RC2 ISO: My coworker tells me, that

/usr/share/atlasstyler/start_AtlasStyler.sh

doesn't have the executable flag set?! This file is extracted from/ created by the atlasstyler.deb package WITH executable flag set?! How could it be, that the flag is getting lost between VM and ISO?

very odd, I see the same thing in 5.0rc2. VM: +x, ISO: -x.

and it is still broken in the 5.0rc3 ISO.

shall we stash a copy of the script in /usr/local/bin/ (and edit the paths within it), and try to point to there.. ?

ie add this to the end of install_atlasstyler.sh:

cat << EOF >> /usr/bin/atlasstyler

if [ $? -ne 0 ] ; then
   . /usr/share/atlasstyler/start_Atlasstyler.sh
fi
EOF

#?or? if [ ! -x /usr/share/atlasstyler/start_Atlasstyler.sh ] ; then

Hamish

comment:7 Changed 8 years ago by hamish

variant of the patch tested in rc3 & it works, will applied in svn r7068.

please apply locally and test.

Hamish

comment:8 Changed 8 years ago by hamish

Keywords: iso added

comment:9 Changed 8 years ago by rouault

I confirm a permission issue on AtlasStyler? with "osgeo-live-5.0-mini.iso 26-Aug-2011 03:12 3.5G"

comment:10 Changed 8 years ago by hamish

fixed the fix in r7108.

comment:11 Changed 8 years ago by darkblueb

  sudo find / -name '*.sh' -ls | grep 'r--'

search for .sh scripts without +x perms

comment:12 Changed 8 years ago by hamish

as per comment:10, this bug was hopefully fixed in r7108.


did the find command find anything? (it wouldn't surprise me if it did)

a theory: it seems the hardlinking of duplicate files need to set all duplicate files as the same permissions? and I don't really know, but maybe it picks the most restrictive to go with. or maybe just the first one it comes to. ?

Hamish

comment:13 Changed 8 years ago by darkblueb

this small program in python will filter a list of files to ones that start with #!

Made to be combined with a file list generator and an action, like examine perms for lack of -x

import os,sys

gUsage = '''
  app filename  RETURNS True for #! False otherwise
'''
if len(sys.argv) < 2:
  print gUsage
  sys.exit(0)

tFName = sys.argv[1]
tF = open( tFName, 'r')
tChars = tF.read(2)

if tChars == '#!':
  print tFName
  sys.exit(1)
else:
  sys.exit(0)

comment:14 Changed 8 years ago by darkblueb

this issue appears to be fixed in RC6

comment:15 Changed 8 years ago by alfonx

Cc: alfonx removed
Resolution: fixed
Status: newclosed

tested on rc6 iso - also works here now. closing as fixed.

Note: See TracTickets for help on using tickets.