-
Notifications
You must be signed in to change notification settings - Fork 806
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 & six.moves not found fixed #179
base: master
Are you sure you want to change the base?
Conversation
Hey @punixcorn, thanks for working on a fix for the reported issue. I had the same problem with
Taking your approach, I added
Thanks! |
Hey @jimdiroffii , thanks for calling that out. I don't seem to have any more syntax errors on my end , but I will apply those changes to my fork. I am glad I was of help to you. |
in lib/utils.py and lib/nasm.py
Hey @punixcorn, there seems to be a small typo in the README you are trying to merge, for debian it's |
@CriimBow i saw these typos, i fixed and applied the changes to my fork but forgot about it here, i will fix that shortly, thanks for calling it out |
i got this error and i don't know if i should make it raw string For help, type "help". |
@L42yH4d3s, yes you should. pattern = re.compile(r'|'.join(JMPCALL).replace(r' ', r'\ ')) |
issue #177 fixes applied
1. fix SyntaxWarning
adding r to the strings or replace
b'...'
withr'...'
, based on the warningeg
to make it raw string, pull request #178 fixes this.
2. fix
six.moves
not foundusing
and remove the
six.py
in thelib/
so it uses the global ones and not the local one found in
lib