-
Notifications
You must be signed in to change notification settings - Fork 109
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
Makefile: implement "fully source containers" HMS-3883 #1704
base: main
Are you sure you want to change the base?
Makefile: implement "fully source containers" HMS-3883 #1704
Conversation
|
||
# Function to search for Python files in a directory | ||
define find_python_files | ||
$(shell find $(1) -type f -exec file {} \; | grep "Python script" | grep -Eo "^[^:]+") |
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.
If this will catch also stage files? They lack .py
extension. There is also stage org.osbuild.tar
which is python file but frequently by script is processed as a tar archive:)
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.
That's the only reason for this not-so-nicce find-grep pipe - to get those files in consideration :-)
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.
Just running in the osbuild directory should show them all...
find . -type f -exec file {} \; | grep "Python script" | grep -Eo "^[^:]+"
(actually I now see that I have ".py" files twice - but that won't hurt make
I guess)
8d94980
to
5c55362
Compare
5c55362
to
cacfca6
Compare
cacfca6
to
9c70e64
Compare
This PR is stale because it has been open 30 days with no activity. Remove "Stale" label or comment or this will be closed in 7 days. |
Getting the stack up and running locally but all from source