View Issue Details

IDProjectCategoryView StatusLast Update
0002140Slicer4Core: Extensionspublic2012-08-21 09:46
Reporterpieper Assigned Tojcfr  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product Version 
Target VersionSlicer 4.1.1Fixed in VersionSlicer 4.1.1 
Summary0002140: settings error when using extension manager
Description

Fresh install of the nightly build of today. See attached image.

TagsNo tags attached.

Relationships

related to 0002053 acknowledgedjcfr qSlicerExtensionsManagerModelTest - Re-activate "readonly" test on windows and MacOSX. 
related to 0002144 closedjcfr Fail to install extension if Launcher settings is read-only 
child of 0002143 closedjcfr Extend Additional Launcher Settings to support user specific settings 

Activities

2012-05-31 05:32

 

jcfr

jcfr

2012-05-31 06:21

administrator   ~0004706

Do you have any idea of what could be wrong ? This is also what I noticed ... I initially thought that the permission on the file were incorrect.

2012-05-31 06:30

 

pieper

pieper

2012-05-31 06:31

administrator   ~0004707

Looks like the file is not writable by users. Maybe the installer sets it up this way...

jcfr

jcfr

2012-05-31 09:45

administrator   ~0004711

Just created a small project allowing to do some experiment. See https://github.com/jcfr/TestCMakeFileInstall

jcfr

jcfr

2012-05-31 12:01

administrator   ~0004712

It resumes that even after explicitly setting PERMISSIONS associated with the install(FILES ...) statement. The installed file is still not writable.

install(
FILES "foo.ini"
DESTINATION bin
RENAME fooInstalled.ini
PERMISSIONS OWNER_READ OWNER_WRITE GROUP_READ GROUP_WRITE WORLD_READ WORLD_WRITE
)

jcfr

jcfr

2012-05-31 13:13

administrator   ~0004713

Since explicitly settings the file permission within the CMake install rule doesn't work as expected, the workaround associated with 4.1.1 release will consist in asking the user to explicitly update the permission associated with the file.

Let's note that an extension should fail to be installed if the launcher settings is read-only. This is captured by issue 0002144 and has been targeted for 4.1.1.

Setting the permissions programmatically has also been considered but this could resume to be problematic on windows 7 platform. See issue 0002053. Given the fact the release if due for June 1st, the manual update of permissions will be suggested.

Finally, the ideal solution would consist in having user specific extension settings differentiated by version of Slicer. This is captured by issue 0002143 and is targeted for 4.2.0.

jcfr

jcfr

2012-06-01 07:36

administrator   ~0004727

Let's also note that setting file permissions has been fixed in 4.8.1. See [1]

I believe the problem could also be avoided by running Slicer as administrator.

[1] https://bugreports.qt-project.org/browse/QTBUG-247

jcfr

jcfr

2012-07-27 15:40

administrator   ~0005384

Dear issue reporter,

Good news :) Slicer developers SOLVED the problem you reported - YOU now need to VERIFY and CLOSE this issue.

Issue History

Date Modified Username Field Change
2012-05-31 05:32 pieper New Issue
2012-05-31 05:32 pieper Status new => assigned
2012-05-31 05:32 pieper Assigned To => jcfr
2012-05-31 05:32 pieper File Added: rdesktop - fisk.bwh.harvard.edu_231.png
2012-05-31 06:21 jcfr Note Added: 0004706
2012-05-31 06:30 pieper File Added: rdesktop - fisk.bwh.harvard.edu_232.png
2012-05-31 06:31 pieper Note Added: 0004707
2012-05-31 09:45 jcfr Note Added: 0004711
2012-05-31 12:01 jcfr Note Added: 0004712
2012-05-31 12:50 jcfr Relationship added parent of 0002143
2012-05-31 12:50 jcfr Relationship deleted parent of 0002143
2012-05-31 12:50 jcfr Relationship added child of 0002143
2012-05-31 12:55 jcfr Relationship added related to 0002053
2012-05-31 13:02 jcfr Relationship added related to 0002144
2012-05-31 13:13 jcfr Note Added: 0004713
2012-05-31 13:13 jcfr Status assigned => resolved
2012-05-31 13:13 jcfr Fixed in Version => Slicer 4.1.1 - June 1st 2012
2012-05-31 13:13 jcfr Resolution open => fixed
2012-06-01 07:36 jcfr Note Added: 0004727
2012-07-27 15:40 jcfr Note Added: 0005384
2012-07-28 10:19 pieper Status resolved => closed
2012-08-21 09:46 jcfr Target Version => Slicer 4.1.1