Hua work setuid#4
Merged
Merged
Conversation
- add install.py (pywrapper-install): automates gcc compile, chmod setuid, symlink creation and setuid_ connector locking (replaces manual steps) - pywrapper.c: replace __FILE__ path with /proc/self/exe for runtime correctness; add execv error handling; use snprintf/strncpy for buffer safety; adopt setuid_<prog> naming convention with dsarch.py fallback for back compat; update setup instructions and convention notes in header comments - pgstart.py: fix PGLOG['USER'] KeyError (use GDEXUSER); replace hardcoded 'zji' with ruser==euser; replace os.system with subprocess.run/Popen; add subprocess import - pyproject.toml: bump to 1.0.4; register pywrapper-install console script Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Display usage via 'more' when pywrapper-install is run with no arguments. Include install.usg in package-data so it is bundled with the distribution. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Consistent with the setuid_ naming convention for connector scripts. pip install now places setuid_pywrapper in bin/ instead of pywrapper.py. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- install.usg: add dependency requirement section (rda_python_setuid must be declared in dependencies and setuid_ entry point must be registered) - README.md: expand from one line to full user guide covering overview, dependency requirement, environment setup (venv and conda), installation sequence, and runtime flow Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
--simple creates PROGRAM -> setuid_PROGRAM symlink directly, skipping the setuid/pywrapper mechanism entirely. The program runs as the current user. --user is not required with --simple. Useful for users who do not need or cannot set up the setuid wrapper. Update install.usg and README accordingly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.