packages; fix last remnants of hard-coded BR2_DL_DIR
cover packages; fix last remnants of hard-coded BR2_DL_DIR
Hello All!
Two packages, asterisk and dahdi-linux, were overlooked when we did the transition to a per-package download directory, which means they are now failing because they can't find their extra files.
This series fixes that.
Regards,
Yann E. MORIN.
The following changes since commit 72703d02b96bb1f7f94f705d3a7199599b1bc980
support/dependencies: check that PATH does not contain CWD (2018-05-09 23:26:18 +0200)
are available in the git repository at:
git://git.buildroot.org/~ymorin/git/buildroot.git
for you to fetch changes up to 7cf1082497a62fa2ea47b0357101efa33b453aac
package/dahdi-linux: use package's DL_DIR (2018-05-11 18:10:20 +0200)
----------------------------------------------------------------
Yann E. MORIN (2):
package/asterisk: use package's DL_DIR
package/dahdi-linux: use package's DL_DIR
package/asterisk/asterisk.mk | 2 +- package/dahdi-linux/dahdi-linux.mk | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------'
PATCH 1/2 package/asterisk: use package's DL_DIR
Sicne we reworked the download infra, the location for packages to look for their files has moved to a per-package directory.
For systems where a download of the asterisk sound files was already done in a version prior to the rework, all was working fine so far, because the files were indeed in the main DL directory.
But for systems where the download is first attempted after the rework, the files are not found (even though they are properly downloaded).
Fix the location where asterisk looks for its extra files.
--- package/asterisk/asterisk.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
package/asterisk/asterisk.mk +1 -1
@@ -102,7 +102,7 @@ ASTERISK_CONF_OPTS = \ --with-libxml2 \ --with-ncurses="$(STAGING_DIR)/usr" \ --with-sqlite3="$(STAGING_DIR)/usr" \ - --with-sounds-cache=$(BR2_DL_DIR) + --with-sounds-cache=$(ASTERISK_DL_DIR) # avcodec are from ffmpeg. There is virtually zero chance this could # even work; asterisk is looking for ffmpeg/avcodec.h which has not
Signature · 1 line
2.14.1
PATCH 1/2 package/asterisk: use package's DL_DIR
Quoted 8 lines
From: "Yann Morin" <[email protected]>
To: "buildroot" <[email protected]>
Cc: "Yann Morin" <[email protected]>, "***** *****" <[email protected]>, "Carlos Santos" <[email protected]>, "Maxime Hadjinlian" <[email protected]>
Sent: Friday, May 11, 2018 1:29:56 PM
Subject: [PATCH 1/2] package/asterisk: use package'sDL_DIRSicne we reworked the download infra, the location for packages to look
Since
[...]
Signature · 3 lines
Carlos Santos (Casantos) - DATACOM, P&D ?Marched towards the enemy, spear upright, armed with the certainty that only the ignorant can have.? ? Epitaph of a volunteer
PATCH 2/2 package/dahdi-linux: use package's DL_DIR
In reply to Yann E. MORINSicne we reworked the download infra, the location for packages to look for their files has moved to a per-package directory.
For systems where a download of the dahdi firmware files was already done in a version prior to the rework, all was working fine so far, because the files were indeed in the main DL directory.
But for systems where the download is first attempted after the rework, the files are not found (even though they are properly downloaded).
Fix the location where dahdi-linux looks for its extra files.
--- package/dahdi-linux/dahdi-linux.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
package/dahdi-linux/dahdi-linux.mk +1 -1
@@ -56,7 +56,7 @@ DAHDI_LINUX_MODULE_MAKE_OPTS = \ define DAHDI_LINUX_EXTRACT_FW $(foreach f,$(DAHDI_LINUX_FW_FILES),\ - cp $(BR2_DL_DIR)/$(f) $(@D)/drivers/dahdi/firmware/$(f)$(sep)) + cp $(DAHDI_LINUX_DL_DIR)/$(f) $(@D)/drivers/dahdi/firmware/$(f)$(sep)) endef DAHDI_LINUX_POST_EXTRACT_HOOKS += DAHDI_LINUX_EXTRACT_FW
Signature · 1 line
2.14.1
PATCH 2/2 package/dahdi-linux: use package's DL_DIR
Quoted 8 lines
From: "Yann Morin" <[email protected]>
To: "buildroot" <[email protected]>
Cc: "Yann Morin" <[email protected]>, "***** *****" <[email protected]>, "Carlos Santos" <[email protected]>, "Maxime Hadjinlian" <[email protected]>
Sent: Friday, May 11, 2018 1:29:57 PM
Subject: [PATCH 2/2] package/dahdi-linux: use package'sDL_DIRSicne we reworked the download infra, the location for packages to look
Since
[...]
Signature · 3 lines
Carlos Santos (Casantos) - DATACOM, P&D ?Marched towards the enemy, spear upright, armed with the certainty that only the ignorant can have.? ? Epitaph of a volunteer
Thomas Petazzoni
Appliedcover packages; fix last remnants of hard-coded BR2_DL_DIR
In reply to Yann E. MORINHello,
Quoted 3 lines · Yann E. MORIN
Yann E. MORIN (2):
package/asterisk: use package'sDL_DIR
package/dahdi-linux: use package'sDL_DIR
Applied, after fixing the "Since" typo in the commit log, as noticed by Carlos.
Thanks!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com