-
Notifications
You must be signed in to change notification settings - Fork 9
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
Updates for renaming script in 726 #797
Conversation
@@ -11,11 +11,6 @@ find . -name control -type f | while IFS= read -r f; do | |||
fi | |||
done | |||
|
|||
# we don't support buster anymore |
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.
Any particular reason to remove it?
@@ -49,43 +50,14 @@ EOF | |||
echo | |||
done | |||
|
|||
# 2. Copyright files | |||
find . -name '*copyright*' -type f | while IFS= read -r f; do |
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.
copyright updates were generating problems?
done | ||
|
||
# 3. Test file if needed | ||
if [[ -f ubuntu/debian/tests/ ]]; then |
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.
I liked the autokpgtest :(
echo | ||
fi | ||
|
||
# 4. Changelog files |
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.
was not working?
find . -name '*ignition*' -type f | while IFS= read -r f; do | ||
echo "Renaming ${f} -> ${f/ignition-/gz-}" | ||
git mv "${f}" "${f/ignition-/gz-}" | ||
done | ||
|
||
# 6. watch files |
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.
Yep watch files are not very useful for us but did they generate any problem?
Closing. Import a subset of this PR in a32c867 |
Following up comment in #726