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

invalid escape sequence s #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

abkrim
Copy link

@abkrim abkrim commented Mar 29, 2024

Correct code

python3 pylaunchd_gui.py
/Users/abkrim/Applications/pylaunchd/pylaunchd_gui.py:379: SyntaxWarning: invalid escape sequence '\s'
  paths = re.findall('^\s+path =\s(.*$)', details, re.MULTILINE)
/Users/abkrim/Applications/pylaunchd/pylaunchd_gui.py:383: SyntaxWarning: invalid escape sequence '\s'
  states = re.findall('^\s+state =\s(.*$)', details, re.MULTILINE)

Best regards

@glowinthedark
Copy link
Owner

can you please provide the output of running sw_vers and python3 -V from the terminal?

@abkrim
Copy link
Author

abkrim commented Apr 4, 2024

sw_vers
ProductName:		macOS
ProductVersion:		14.4.1
BuildVersion:		23E224
python3 -V
Python 3.12.2

@glowinthedark
Copy link
Owner

glowinthedark commented Apr 4, 2024

thanks @abkrim! haven't yet validated it to work as expected on sonoma. Apple do mention that the output of launchctl should not be relied on and that the shape of the data might change in future macos versions, and it looks that this is what happened here; it will take additional investigation to make sure it works with both the new and previous macos versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants