-
Notifications
You must be signed in to change notification settings - Fork 2k
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
pkg/openwsn: prevent build loop #5251
Conversation
I could also remove the target instead of just the command to execute, if desired. |
@@ -5,6 +5,5 @@ PKG_VERSION=ff25e5d0ae5d344ed793a724d60532fb917bf1f8 | |||
.PHONY: all | |||
|
|||
all: git-download | |||
"$(MAKE)" -C $(PKG_DIR) |
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.
Hm, but that line should actually build the package. It is wrong and should be $(MAKE)" -C $(PKG_BUILDDIR)
.
If you remove it, does the openwsn application build at all?
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.
addressed
179031f
to
1c73e7a
Compare
@kaspar030 you were right, it didn't build openwsn. I fixed it the way you proposed and amended the change. It now attempts to build openwsn (fails for other reasons, though) |
I included this bug fix into the current release, because it is .. a bug fix. However, the severity may be very low, because we currently have no working examples/applications utilizing openwsn. |
Anyone care to acknowledge? |
1 similar comment
Anyone care to acknowledge? |
ACK and go. |
When I build the
openwsn
application from the application repository, then I get a build loop.A smiliar line is in
pkg/Makefile.git
, so I guess it is not needed to repeat that inpkg/openwsn/Makefile