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

don't build openwrt SDK #337

Closed
booo opened this issue Mar 9, 2016 · 10 comments
Closed

don't build openwrt SDK #337

booo opened this issue Mar 9, 2016 · 10 comments

Comments

@booo
Copy link
Member

booo commented Mar 9, 2016

At the moment we build the openwrt SDK on each build. I never used the SDK and as far as I know it's also no dependency for something we need. I suggest that we don't build the SDK anymore.

@SvenRoederer
Copy link
Contributor

Atm. the only reason for the SDK might be, to use it to build the tunnelbroker for the VPN experiments.

@booo
Copy link
Member Author

booo commented Mar 10, 2016

Who does this?

@SvenRoederer
Copy link
Contributor

Who knows ... I can just imagine this as a valid reason for using the SDK.

Maybe it's a good idea to stop not build the SDK in unstable build and only in the final stable build. So in case anyone likes to compile something for released stable, he can use the SDK for this.

@booo
Copy link
Member Author

booo commented Mar 10, 2016

Have you ever tried to build something with the SDK? Do you know if it's really working? If they want to build something they can also use the normal make process.

@SvenRoederer
Copy link
Contributor

what about building the SDK only on stable releases? so interested people can just use it without compiling all themselves?

This requires just a test in the Makefile for the build-type stable/unstable (depending on the release-name, as done by the build-bots), and the Makefile can change this option in openwrt/.config

@FreifunkUFO
Copy link

SDK is needed when compiling/ updating packages.. and for freifunk-berlin-unknown packages! so please build SDK with your firmwares!

@andrenarchy
Copy link
Member

I also use the SDK sometimes. So +1 for keeping it. :)

@booo
Copy link
Member Author

booo commented May 19, 2016

Can someone outline the process of building something with the sdk? If it really works we could maybe improve the whole build process with this.

@lynxis
Copy link
Member

lynxis commented May 29, 2016

+1 for keeping it.
@booo see lede-project use it in their 2 phase build. first build only the base system and second build all packages using the sdk.
https://git.lede-project.org/?p=buildbot.git;a=blob;f=phase2/master.cfg;h=e61fce55b5dd6d0f5f2ea38a1b15fbb91e05d995;hb=HEAD

@FreifunkUFO
Copy link

@booo please see http://wiki.openwrt.org/doc/howto/obtain.firmware.sdk
and try it, f.i. with umurmur-openssl or cjdns..

@lynxis lynxis closed this as completed Aug 8, 2016
SvenRoederer added a commit that referenced this issue Jan 8, 2018
03a4dbf Merge pull request #337 from ecsv/batadv-2017.4
9abfeba mcproxy: update to latest version (#339)
d3b2d83 bmx7_json: Fix broken include of json.h
5e96695 nodogsplash2: change configuration defaults (#341)
679f780 bmx: Use repos from https://github.com/bmx-routing and remove bmx7 json-c patch
467f0b6 batman-adv: Fix missing function name for timer_setup
SvenRoederer added a commit that referenced this issue Feb 20, 2018
03a4dbf Merge pull request #337 from ecsv/batadv-2017.4
9abfeba mcproxy: update to latest version (#339)
d3b2d83 bmx7_json: Fix broken include of json.h
5e96695 nodogsplash2: change configuration defaults (#341)
679f780 bmx: Use repos from https://github.com/bmx-routing and remove bmx7 json-c patch
467f0b6 batman-adv: Fix missing function name for timer_setup
SvenRoederer added a commit to SvenRoederer/freifunk-berlin-firmware that referenced this issue Mar 4, 2018
03a4dbf Merge pull request freifunk-berlin#337 from ecsv/batadv-2017.4
9abfeba mcproxy: update to latest version (freifunk-berlin#339)
d3b2d83 bmx7_json: Fix broken include of json.h
5e96695 nodogsplash2: change configuration defaults (freifunk-berlin#341)
679f780 bmx: Use repos from https://github.com/bmx-routing and remove bmx7 json-c patch
467f0b6 batman-adv: Fix missing function name for timer_setup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants