Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added a resolution case on Fedora 37+ #7114

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Villafruela
Copy link

Added a check-list of actions to solve the execution of Slicer on Fedora 37+

Added a check-list of actions to solve the execution of Slicer on Fedora 37+
Copy link
Member

@pieper pieper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for suggesting these changes 👍

We want the documentation to be complete and helpful. Since differences across linux versions can be a complex topic we want to be sure we are specifically addressing issues in the correct spot (e.g. what's unique to Fedora vs what's generic to linux).

Comment on lines 126 to 129
If you double-clicked on the app and it doesn't launch, you can try this (in order):
- verify execution rights of the app (Right click on the app > properties > executable as a program checked) or in terminal, in the app folder:

chmod +x Slicer
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be needed since the file will be executable after extracting from the tar file. Did you ever have it happen that this was not the case?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some unzip software have special usage. Like Info-Zip 3.0 for instance, where you need to unzip as root if you want to restore the rights scheme on the files. Some also configure their unzip software to restore files only in 666 (rw-rw-rw-) maximum for security reasons. So you can't pass this option to troubleshoot i think (or at least you have to check that permissions are 755...)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, that makes more sense. Maybe you can make a small edit to the documentation text summarizing this.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure i'll be happy to...

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you be interested in a french translation of the documentation ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you be interested in a french translation of the documentation ?

Yes! In fact, there's a project devoted to that and any help would be greatly appreciated!

There is an internationalization group and infrastructure that you can learn about here:

https://projectweek.na-mic.org/PW39_2023_Montreal/Projects/3DSlicerInternationalization/

https://chanzuckerberg.com/eoss/proposals/3d-slicer-in-my-language-internationalization-and-usability-improvements/

The first fully-French tutorial was recently held, but there is a lot more to be done:

https://sliceriml.github.io/

@spujol is the PI of these projects.

Comment on lines 130 to 133
- launch the app from terminal :

./Slicer
- if you get the "error: Failed to obtain launcher executable name !" message : move the app in your Software Home folder and double click on the app, it should launch.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would help if you could provide more context for this. Is there something special on Fedora that files are downloaded by default to a place where they can't be executed?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On fedora, only User's Home folders get execution rights. But for some reason, i do not know why but i experienced it : if you unpack the files in the download folder : you get the error message, if you unpack in the "Applications" folder, it just work fine... That's why i suggested the addition to the documentation.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This also makes sense and would be nice to mention in the text. Maybe some documentation about installing applications in these systems.

Villafruela added a commit to Villafruela/Slicer that referenced this pull request Sep 1, 2023
following Slicer#7114 modification of troubleshooting for the Fedora distro
Copy link
Member

@pieper pieper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@pieper pieper enabled auto-merge (squash) September 1, 2023 13:58
following Slicer#7114 modification of troubleshooting for the Fedora distro (rights  for execution)
auto-merge was automatically disabled September 4, 2023 08:09

Head branch was pushed to by a user without write access

@jcfr jcfr added the Type: Documentation Issues regarding documentation label Nov 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Issues regarding documentation
Development

Successfully merging this pull request may close these issues.

None yet

3 participants