Skip to content
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_SOURCE_LOCAL override is broken #13055

Closed
haukepetersen opened this issue Jan 9, 2020 · 4 comments · Fixed by #13056
Closed

PKG_SOURCE_LOCAL override is broken #13055

haukepetersen opened this issue Jan 9, 2020 · 4 comments · Fixed by #13056
Assignees
Labels
Area: build system Area: Build system Area: pkg Area: External package ports Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Comments

@haukepetersen
Copy link
Contributor

Description

using a local copy for a package with PKG_SOURCE_LOCAL is broken since #13036 was merged. Now the packages source is copied from the local path, but it is not build anymore.

This should be fixed ASAP, as this is stalling my ongoing experiments :-)

Steps to reproduce the issue

Take any package (e.g. nimble) and try to build RIOT with a local copy of that package

PKG_SOURCE_LOCAL_NIMBLE=/local/path/foo make clean all

You will see in the build output, that the package is actually copied:

Building application "nimble_gatt" for "nrf52dk" with MCU "nrf52".

rm -Rf /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/pkg/nrf52dk/nimble
mkdir -p $(dirname /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/pkg/nrf52dk/nimble)
cp -a /home/hauke/dev/ble/mynewt-nimble /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/pkg/nrf52dk/nimble
touch /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/pkg/nrf52dk/nimble/.prepared
"make" -C /home/hauke/dev/riot/RIOT/pkg/nimble
"make" -C /home/hauke/dev/riot/RIOT/boards/nrf52dk
"make" -C /home/hauke/dev/riot/RIOT/boards/common/nrf52xxxdk
"make" -C /home/hauke/dev/riot/RIOT/core
...

but it is not compiled

...
"make" -C /home/hauke/dev/riot/RIOT/sys/xtimer
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_controller.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_drivers_nrf5x.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_host.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_host_store_ram.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_host_util.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_npl_riot.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_porting_nimble.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_riot_contrib.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_svc_gap.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_svc_gatt.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_tinycrypt.a: No such file or directory
arm-none-eabi-gcc: error: /home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_transport_ram.a: No such file or directory
/home/hauke/dev/riot/RIOT/examples/nimble_gatt/../../Makefile.include:537: recipe for target '/home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_gatt.elf' failed
make: *** [/home/hauke/dev/riot/RIOT/examples/nimble_gatt/bin/nrf52dk/nimble_gatt.elf] Error 1

Expected results

Should be build using the local copy

Actual results

does not

@haukepetersen haukepetersen added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: build system Area: Build system Area: pkg Area: External package ports labels Jan 9, 2020
@fjmolinas fjmolinas self-assigned this Jan 9, 2020
@fjmolinas
Copy link
Contributor

Ok, this slipped in, we did not test this. Will work on fix.

@haukepetersen
Copy link
Contributor Author

Thanks!

@fjmolinas
Copy link
Contributor

This fixes compile

diff --git a/pkg/local.mk b/pkg/local.mk
index 4f5755da2e..b55aba3c8c 100644
--- a/pkg/local.mk
+++ b/pkg/local.mk
@@ -8,6 +8,8 @@
 
 .PHONY: prepare clean
 
+all: $(PKG_BUILDDIR)/.prepared
+
 prepare: $(PKG_BUILDDIR)/.prepared
        @true
 

Trying to test how it clean.s

@haukepetersen
Copy link
Contributor Author

I can verify, that works.

Are you going to open the according PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: build system Area: Build system Area: pkg Area: External package ports Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants