Skip to content

jansson: Update to v2.10#4289

Merged
hnyman merged 1 commit intoopenwrt:masterfrom
Tiboris:jansson-update
Apr 22, 2017
Merged

jansson: Update to v2.10#4289
hnyman merged 1 commit intoopenwrt:masterfrom
Tiboris:jansson-update

Conversation

@Tiboris
Copy link
Contributor

@Tiboris Tiboris commented Apr 21, 2017

Maintainer: @kissg1988
Compile tested: (mips, TL-WR842N, Chaos Calmer, r49474)

Signed-off-by: Tibor Dudlák tibor.dudlak@gmail.com

Description: jansson in version 2.9 or later is required as dependency for jose

@diizzyy
Copy link
Contributor

diizzyy commented Apr 21, 2017

Use bz2 tarball instead and PKG_HASH

@kissg1988
Copy link
Contributor

Approved from my side. Thanks for the update!

@hnyman
Copy link
Contributor

hnyman commented Apr 22, 2017

@Tiboris
You need amend the commit so that you properly sign-off in the commit message that gets stored to the git history (unlike this PR discussion where your sign-off is now).

Please also follow the feedback from diizzyy and switch to smaller tarball and use sha256-based PKG_HASH instead of PKG_MD5SUM.

You have the commit in a branch, so amending it should be straight-forward.

jansson in version 2.9 or later is required as dependency for jose

Signed-off-by: Tibor Dudlák <tibor.dudlak@gmail.com>
@Tiboris
Copy link
Contributor Author

Tiboris commented Apr 22, 2017

Thank you all @kissg1988 @hnyman @diizzyy, changes should be now acceptable. 😄

@hnyman hnyman merged commit 74a02e2 into openwrt:master Apr 22, 2017
@Tiboris Tiboris deleted the jansson-update branch April 22, 2017 12:38
@Tiboris
Copy link
Contributor Author

Tiboris commented May 6, 2017

Hi, could you please help me with configuring my device?
I am unable to get this latest package.

Is there any mirror where latest packages are located on?

Thank you very much for help.

opkg update | grep http | uniq | grep gz | cut -d ' ' -f2
http://downloads.openwrt.org/chaos_calmer/15.05.1/ar71xx/generic/packages/base/Packages.gz.
http://downloads.openwrt.org/chaos_calmer/15.05.1/ar71xx/generic/packages/luci/Packages.gz.
http://downloads.openwrt.org/chaos_calmer/15.05.1/ar71xx/generic/packages/management/Packages.gz.
http://downloads.openwrt.org/chaos_calmer/15.05.1/ar71xx/generic/packages/packages/Packages.gz.
http://downloads.openwrt.org/chaos_calmer/15.05.1/ar71xx/generic/packages/routing/Packages.gz.
http://downloads.openwrt.org/chaos_calmer/15.05.1/ar71xx/generic/packages/telephony/Packages.gz.
http://dl.eko.one.pl/chaos_calmer/ar71xx/packages/Packages.gz.

@kissg1988
Copy link
Contributor

@Tiboris I believe your question should be posted to the support forums but anyway, new or updated packages are very rarely added to the repositories of already released versions, except if your change is about fixing a bug or a security flaw. Not sure about the exact workflow here but basically you'd need to push your modifications to the "for-15.05" branch to see them appearing in the official Chaos Calmer repositories. However, I don't think this particular change would be accepted as it's neither a bugfix nor a security fix.

Please see CONTRIBUTING.md.

If you build OpenWrt against the master branch, the file /etc/opkg/distfeeds.conf within the output image will contain links pointing to the snapshot repositories which most of the time holds the most up-to-date packages from the master branch - except if Buildbot is down or has some issues building the packages for a specific architecture.

Please build and install the package manually if you intend to use it on your device. You might report the issue with Buildbot to the developer forums or to the openwrt-devel mailing list.

Hope this helps!

@Tiboris Tiboris mentioned this pull request May 7, 2017
2 tasks
@kissg1988
Copy link
Contributor

@Tiboris looks like your device has a NOR flash so the correct repository to use would be this one.

Install the latest build from here on your device (hopefully it would boot up fine) and then build and install jansson by hand or as a dirty hack, you might install it from the ar71xx.nand repository as it already contains the updated version of jansson.

@Tiboris Tiboris restored the jansson-update branch October 15, 2017 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants