-
Notifications
You must be signed in to change notification settings - Fork 440
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
Use the proper requirements.txt path #707
Use the proper requirements.txt path #707
Conversation
fix the implementation to look for requirements.txt in the part's source dir. LP: #1588855 Signed-off-by: Sergio Schvezov <sergio.schvezov@ubuntu.com>
Nice, looks good 👍 |
@@ -11,7 +11,9 @@ confinement: strict | |||
parts: | |||
python2: | |||
plugin: python2 | |||
source: . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm... should this be required, now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well no, I guess not since we have python-packages
, but could we toss an obvious error if requirements
is used without source
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kyrofa I plan to solve than when I deal with the culprit for https://bugs.launchpad.net/snapcraft/+bug/1606894
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, very good.
LP: #1588855 Signed-off-by: Sergio Schvezov <sergio.schvezov@ubuntu.com>
fix the implementation to look for requirements.txt in the
part's source dir.
LP: #1588855
Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com