-
Notifications
You must be signed in to change notification settings - Fork 991
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
Fix error in replacement for RUNTIME_NAME #993
Labels
Projects
Comments
c3d
added
enhancement
Improvement to an existing feature
needs-review
Needs to be assessed by the team.
labels
Oct 19, 2020
c3d
added
bug
Incorrect behaviour
no-backport-needed
and removed
enhancement
Improvement to an existing feature
labels
Oct 19, 2020
c3d
added a commit
to c3d/kata-containers
that referenced
this issue
Oct 19, 2020
In commit 966bd57 for PR kata-containers#902, the makefile was changed to automate the replacement of user variables. However, one variable was treated specially in the original `sed` replacements, namely `RUNTIME_NAME` which was replaced by `$(TARGET)`. This commit adds the `RUNTIME_NAME` variable to the makefile in order to ensure that the replacement works correctly. Fixes: kata-containers#993 Signed-off-by: Christophe de Dinechin <dinechin@redhat.com>
c3d
added a commit
to c3d/kata-containers
that referenced
this issue
Oct 19, 2020
In commit 966bd57 for PR kata-containers#902, the makefile was changed to automate the replacement of user variables. However, one variable was treated specially in the original `sed` replacements, namely `RUNTIME_NAME` which was replaced by `$(TARGET)`. This commit adds the `RUNTIME_NAME` variable to the makefile in order to ensure that the replacement works correctly. Fixes: kata-containers#993 Signed-off-by: Christophe de Dinechin <dinechin@redhat.com>
c3d
added a commit
to c3d/kata-containers
that referenced
this issue
Oct 19, 2020
In commit 966bd57 for PR kata-containers#902, the makefile was changed to automate the replacement of user variables. However, one variable was treated specially in the original `sed` replacements, namely `RUNTIME_NAME` which was replaced by `$(TARGET)`. This commit adds the `RUNTIME_NAME` variable to the makefile in order to ensure that the replacement works correctly. Fixes: kata-containers#993 Signed-off-by: Christophe de Dinechin <dinechin@redhat.com>
c3d
added a commit
that referenced
this issue
Apr 12, 2021
In commit 966bd57 for PR #902, the makefile was changed to automate the replacement of user variables. However, one variable was treated specially in the original `sed` replacements, namely `RUNTIME_NAME` which was replaced by `$(TARGET)`. This commit adds the `RUNTIME_NAME` variable to the makefile in order to ensure that the replacement works correctly. Fixes: #993 Signed-off-by: Christophe de Dinechin <dinechin@redhat.com>
c3d
added a commit
to c3d/kata-containers
that referenced
this issue
Apr 12, 2021
In commit 966bd57 for PR kata-containers#902, the makefile was changed to automate the replacement of user variables. However, one variable was treated specially in the original `sed` replacements, namely `RUNTIME_NAME` which was replaced by `$(TARGET)`. This commit adds the `RUNTIME_NAME` variable to the makefile in order to ensure that the replacement works correctly. Fixes: kata-containers#993 Signed-off-by: Christophe de Dinechin <dinechin@redhat.com>
c3d
added a commit
to c3d/kata-containers
that referenced
this issue
Apr 27, 2021
In commit 966bd57 for PR kata-containers#902, the makefile was changed to automate the replacement of user variables. However, one variable was treated specially in the original `sed` replacements, namely `RUNTIME_NAME` which was replaced by `$(TARGET)`. This commit adds the `RUNTIME_NAME` variable to the makefile in order to ensure that the replacement works correctly. Fixes: kata-containers#993 Fixes: kata-containers#1164 Signed-off-by: Christophe de Dinechin <dinechin@redhat.com> (cherry-picked from 96a4ed7)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In PR #902, specifically 966bd57, there is a problem in the replacement variables, which leads to generating a bad
kata-collect.sh
.This was found while investigating CI failures on the backport version of the fix. See kata-containers/runtime#3005 (comment).
The problem is that in the old code,
RUNTIME_NAME
was replaced in a special way: it was replaced by$(TARGET)
and not$(RUNTIME_NAME)
. The solution is to modify the makefile to setRUNTIME_NAME
toTARGET
.A regexp search confirmed that this is the only such case remaining (I had found another one earlier), although the same regexp identified a typo in one of the replacements before the commit (that is fixed by the commit).
The text was updated successfully, but these errors were encountered: