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

Compiling with scons-3.0.0 fails #258

Closed
Awerick opened this issue Nov 2, 2017 · 6 comments
Closed

Compiling with scons-3.0.0 fails #258

Awerick opened this issue Nov 2, 2017 · 6 comments
Labels

Comments

@Awerick
Copy link

Awerick commented Nov 2, 2017

When using scons version 3, compiling rmlint fails:

$ scons config
[...]
Checking for C header file sys/utsname.h... yes
  File ".../rmlint/src/SConscript", line 29
    env.Alias('install', env.InstallPerm('$PREFIX/bin', programs, 0755))
                                                                     ^
SyntaxError: invalid token

Probably the reason is that scons 3 made the switch to Python 3 and rmlint's build script is Python 2.
(In this particular case the reason is probably: Leading zeros are not allowed in Python 3.)

Fedora 27 Beta only provides scons 3.0.0 but after manually downloading and installing scons 2.5.1, the compilation succeeded.

@sahib sahib added the bug label Nov 3, 2017
@sahib
Copy link
Owner

sahib commented Nov 3, 2017

Hey @Awerick,

...thanks for the hint. Should be corrected as of 11fb8f9.

@SeeSpotRun
Copy link
Collaborator

SeeSpotRun commented Nov 3, 2017

@sahib Is that going to work with archaic distro's (pre python 2.6, e.g. debian lenny?)

@Bfgeshka
Copy link

Bfgeshka commented Nov 3, 2017

@sahib Commit did not fix the issue.

@sahib
Copy link
Owner

sahib commented Nov 3, 2017

@sahib Is that going to work with archaic distro's (pre python 2.6, e.g. debian lenny?)

I'm not sure to be honest. I changed it to int("644", 8) to be completely sure about that (7b93e80).

@sahib Commit did not fix the issue.

Same problem? What python/scons version are you using?

@Bfgeshka
Copy link

Bfgeshka commented Nov 4, 2017

dev-util/scons-3.0.0, python versions are 2.7.14 and 3.6.3.

By the way, sorry to mislead you. I was looking for changes in master and didn't even look into branches.

Current develop builds just fine.

@sahib
Copy link
Owner

sahib commented Nov 5, 2017

Current develop builds just fine.

Okay, good. I'll close this issue then for now.

@sahib sahib closed this as completed Nov 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants