-
Notifications
You must be signed in to change notification settings - Fork 27
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
Configure failed with "file setup.py not found" #163
Comments
Signed-off-by: Steven Esser <sesser@nexb.com>
@PatrickSchloesser Good catch, this is definitely a bug and an easy fix. See the PR I have just opened for the fix: #164 Once all the CI checks have passed, ill merge it in. |
Perfect! Thanks for looking into this. |
Fix paths in root-level deltacode scripts #163
@PatrickSchloesser #164 merged. Will close this ticket after your confirmation. Thanks for the bug report! :) |
I tried running deltacode again with your new changes incorporated, and I'm still getting the same error. We pull anew directly from github each time the license checker is run, so I'm certain I'm using the newest version. I don't think that your fix is incorrect but rather that my python is setup slightly differently than I thought. In any case, we have decided to roll back to the last version of deltacode that worked for us since we have decided to pick a particular commit hash for both scancode and deltacode to prevent changes to the tools causing potential negative impacts. However, when I checkout any commit including or prior to January 24th, the main deltacode script is not included in the topmost directory. We've been running ./deltacode to run deltacode from the top directory since last October, so I'm confused why it disappears when I checkout old commits. Do you have any ideas about what I might be doing incorrectly? Currently, when I checkout the commit from January 24th, the topmost directory includes only: AUTHORS.rst I realize this is not the response you were looking for; however, I do think this particular issue is continuing to happen due to the complexities of how we're running deltacode (randomly assigned linux machine, builds our license checker, pulls deltacode into the license checker src) and not a bug on your end. |
@PatrickSchloesser Thanks for the feedback. Let me check into this on my side. At first glance, I think the missing top-level script come from the way we integrate our repo skeleton: https://github.com/nexB/skeleton. So when you go to check out a "random" commit, it may be files from only this skeleton side. If possible, you may want to revert to using the However, if you are using the very latest from the develop branch, this script should be present and working. What command(s) do you use to fetch the deltacode repo? |
I've simply been using the following commands: What I've observed is that the when I first pull deltacode, the script is there, but as soon as I checkout the old commit, it disappears from the directory. I wasn't aware of the v1.0.0 tag. Is that simply including -v1.0.0 in the run command? |
To checkout the What happens if you omit the |
I see I'll give it a try with the tag. If I do not include the git checkout command, I get the same file "setup.py" not found issue with the same log as in my first post on this issue. I also have the commit hash print with every run, and when I don't specify the git checkout commithash, it shows that it's running with the current head bf767c2. |
It looks like deltacode is fixed by using the v1.0.0 tag you recommended; however, when running the entire process (scancode 3.2.3 -> deltacode), I get the same error as was reported by this issue: #122 . I'm aware that this issue was fixed, but since I'm using an earlier v1.0.0, it's not included. I would simply make the code change locally, but because because deltacode is pulled each time the license checker is run, it would be pretty inelegant to change "tool_version" to "scancode_version" each run on the fly. Do you have any ideas on how to get around this issue? |
@PatrickSchloesser sorry for the delayed response. The only workaround I can think of at the moment would be to run This commit is immediately after merging the fix to #122, so it should be far enough back in history to not have some of the weirdness you experienced when checking out a commithash previously. |
@MaJuRG how about having a docker image for deltacode? |
After downloading deltacode and running ./deltacode --help to configure, I get the following error message:
ERROR: File "setup.py" not found. Directory cannot be installed in editable mode: /local/worker/testruns/1d3c4735-714d-4292-909b-69854d0698ab/src/LicenseChecker ./deltacode/deltacode: line 114: /local/worker/testruns/1d3c4735-714d-4292-909b-69854d0698ab/src/LicenseChecker/deltacode/bin/deltacode: No such file or directory /local/worker/testruns/1d3c4735-714d-4292-909b-69854d0698ab/src/LicenseChecker/bin/process_delta.rb:59:in
read': No such file or directory @ rb_sysopen - /local/worker/testruns/1d3c4735-714d-4292-909b-69854d0698ab/src/LicenseChecker/output.json (Errno::ENOENT)from /local/worker/testruns/1d3c4735-714d-4292-909b-69854d0698ab/src/LicenseChecker/bin/process_delta.rb:59:in
<main>'
I am fairly certain that the "setup.py not found" error is coming from
pip install -e .[testing]
in the configure script: https://github.com/nexB/deltacode/blob/develop/configure#L38, but I'm not sure how to fix it.I am running this on linux and have both python3.7 and python2.7 installed at the level of the root directory.
I realize this is likely not an issue specifically with deltacode; however, I haven't found success finding resources to help elsewhere, so I'd appreciate any help you can provide!
The text was updated successfully, but these errors were encountered: