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

Multiple users cannot open processing due to folder in /tmp/ #666

Closed
jonalf opened this issue Feb 10, 2023 · 9 comments
Closed

Multiple users cannot open processing due to folder in /tmp/ #666

jonalf opened this issue Feb 10, 2023 · 9 comments
Labels
High Priority Very high priority and would like to fix ASAP Linux Issue is specific to Linux

Comments

@jonalf
Copy link

jonalf commented Feb 10, 2023

Description

I maintain a classroom of ubuntu 20.0.4 computers, multiple users, with unique accounts, per day need to be able to launch and run processing. Students started to complain that they were getting a strange error when trying to launch processing. Processing is installed locally in
/usr/local/bin/

After investigating, I discovered that processing creates a directory in:
/tmp/processing
This directory is owned by the user that opened processing, but remains after processing is closed and the user has logged out. I believe that the existence of this directory is keeping other users from launching processing. Rebooting the computer (or just removing the directory in /tmp does allow a new user to start processing.

@benfry
Copy link
Owner

benfry commented Feb 10, 2023

This should be fixed since 4.0.2: #549

Are you using the latest release?

@benfry benfry added the Linux Issue is specific to Linux label Feb 10, 2023
@jonalf
Copy link
Author

jonalf commented Feb 13, 2023

I'm using processing 4.1.2 revision 1290.

I was wondering why we hadn't had this problem earlier in the year. After your comment about the version I checked back on it and discovered that a colleague had upgraded to the newer version last week. The upgrade coincides with the problem occurring.

@benfry
Copy link
Owner

benfry commented Feb 14, 2023

Got it; will check into it—I suspect an extra subfolder may be created that's causing the issue.

@benfry benfry added the High Priority Very high priority and would like to fix ASAP label Feb 14, 2023
@benfry
Copy link
Owner

benfry commented Feb 15, 2023

Can you give it a try with this: https://download.processing.org/processing-230214-linux-x64.tgz

First delete the /tmp/processing folder, but when you run this version, it should hopefully set permissions so that others can share the directory.

benfry added a commit that referenced this issue Feb 15, 2023
@jonalf
Copy link
Author

jonalf commented Feb 16, 2023

First, thanks for the prompt work on this.

The new processing binary gives the folder in tmp 775 permissions. So users in the same group can run processing, but not any user. See below:

$ ls -l /tmp | grep proce
drwxrwxr-x  3 jadw         jadw      4096 Feb 16 11:39 processing

@benfry
Copy link
Owner

benfry commented Feb 16, 2023

Nuts; it's supposed to be doing 777 (via b014874 and 9fed8a2) but perhaps failing silently.

I'll have to do more testing…

benfry added a commit that referenced this issue Feb 16, 2023
@benfry
Copy link
Owner

benfry commented Feb 17, 2023

Ok, here's another to try: https://download.processing.org/processing-230216-linux-x64.tgz

If that doesn't work, is there anything set with the students' user accounts (i.e. umask related) that prevents them from setting a folder to be readable by all? On a Linux testing machine here, even the previous version was properly showing rwxrwxrwx for the folder.

@jonalf
Copy link
Author

jonalf commented Feb 17, 2023

It works! After looking back at the problem it did seem as if it were umask related. In any event, permissions look good now.

$ ls -l /tmp/ | grep proce
drwxrwxrwx  3 jadw           jadw      4096 Feb 16 20:48 processing

@benfry
Copy link
Owner

benfry commented Feb 17, 2023

Awesome, thanks for letting me know. I'll try to get this posted as 4.2 (which also includes a bunch of updates for Python Mode) in the next day or two.

@benfry benfry closed this as completed Feb 17, 2023
benfry added a commit that referenced this issue Feb 17, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
High Priority Very high priority and would like to fix ASAP Linux Issue is specific to Linux
Projects
None yet
Development

No branches or pull requests

2 participants