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

glibc: update to glibc-2.25 #1616

Merged
merged 8 commits into from May 12, 2017

Conversation

Projects
None yet
4 participants
@MilhouseVH
Contributor

MilhouseVH commented May 9, 2017

Continuing discussion from #1612 (comment)

glibc-2.25 changelog: https://sourceware.org/ml/libc-alpha/2017-02/msg00079.html

The trailing colon added to LD_LIBRARY_PATH is breaking the glibc build. Also llvm will create unusable llvm-config-host causing mesa to fail. These issues were first raised in #1086 (thanks @InuSasha).

Question is: should we simply unset LD_LIBRARY_PATH, is prefixing with $TOOLCHAIN/lib required at all? More testing needed...

@MilhouseVH

This comment has been minimized.

Show comment
Hide comment
@MilhouseVH

MilhouseVH May 9, 2017

Contributor

I'll test this PR with this additional commit: http://sprunge.us/dPVR

If that works OK, then I'll include it in this PR.

Contributor

MilhouseVH commented May 9, 2017

I'll test this PR with this additional commit: http://sprunge.us/dPVR

If that works OK, then I'll include it in this PR.

@vpeter4

This comment has been minimized.

Show comment
Hide comment
@vpeter4

vpeter4 May 9, 2017

Contributor

I see you also unset LD_LIBRARY_PATH (I'm still building with). For this to be sure it is ok we need to build all projects from scratch. Maybe something would pop up :)

Contributor

vpeter4 commented May 9, 2017

I see you also unset LD_LIBRARY_PATH (I'm still building with). For this to be sure it is ok we need to build all projects from scratch. Maybe something would pop up :)

@MilhouseVH

This comment has been minimized.

Show comment
Hide comment
@MilhouseVH

MilhouseVH May 9, 2017

Contributor

Yes, I've successfully built RPi/RPi2/Generic with 358aac5, so that's working. Now I'm going to clean build all projects with 5b81782 to see if we need LD_LIBRARY_PATH at all. Should know in about 9 hours...

Contributor

MilhouseVH commented May 9, 2017

Yes, I've successfully built RPi/RPi2/Generic with 358aac5, so that's working. Now I'm going to clean build all projects with 5b81782 to see if we need LD_LIBRARY_PATH at all. Should know in about 9 hours...

@InuSasha

This comment has been minimized.

Show comment
Hide comment
@InuSasha

InuSasha May 9, 2017

Member

the Addons should built, too.
Actual, i make experiments with gcc7 & glibc2.15. It looks like, same addon dependencies need updates, too.
But i can not remeber in detail, if it depend on gcc or glibc update (i am on work).
https://github.com/InuSasha/LibreELEC.tv/tree/feature/gcc7
Status: Generic - LE build; some addon fixed, but not all.

Member

InuSasha commented May 9, 2017

the Addons should built, too.
Actual, i make experiments with gcc7 & glibc2.15. It looks like, same addon dependencies need updates, too.
But i can not remeber in detail, if it depend on gcc or glibc update (i am on work).
https://github.com/InuSasha/LibreELEC.tv/tree/feature/gcc7
Status: Generic - LE build; some addon fixed, but not all.

@MilhouseVH

This comment has been minimized.

Show comment
Hide comment
@MilhouseVH

MilhouseVH May 10, 2017

Contributor

All the projects (RPi, RPi2, Generic, imx6, Odroid_C2, WeTek_*) have built for me with LE master + #1612 + #1616, so dropping/unsetting "LD_LIBRARY_PATH" doesn't seem to be a problem.

I've run-time tested RPi/RPi2/Generic, and no obvious issues.

I also rebuilt the same addons I tested in #1612, and no new issues:

Generic::

vdr-addon, tvheadend42, moonlight, screensaver.shadertoy, visualization.projectm, visualization.shadertoy, visualization.spectrum, visualization.waveform

RPi/RPi2:

vdr-addon, tvheadend42, screensaver.shadertoy, visualization.shadertoy, visualization.spectrum, visualization.waveform

I've updated the libnfs patch using the upstream fix sahlberg/libnfs@5b28ab5

@InuSasha: Hopefully with these updates (PR1612+PR1616) the jump to gcc-7.1 should be less painful, though maybe wait for gcc-7.2?

There's a gcc-6.3 bump available which might be another option.

If anyone wants to update gcc in a separate PR, feel free!

Contributor

MilhouseVH commented May 10, 2017

All the projects (RPi, RPi2, Generic, imx6, Odroid_C2, WeTek_*) have built for me with LE master + #1612 + #1616, so dropping/unsetting "LD_LIBRARY_PATH" doesn't seem to be a problem.

I've run-time tested RPi/RPi2/Generic, and no obvious issues.

I also rebuilt the same addons I tested in #1612, and no new issues:

Generic::

vdr-addon, tvheadend42, moonlight, screensaver.shadertoy, visualization.projectm, visualization.shadertoy, visualization.spectrum, visualization.waveform

RPi/RPi2:

vdr-addon, tvheadend42, screensaver.shadertoy, visualization.shadertoy, visualization.spectrum, visualization.waveform

I've updated the libnfs patch using the upstream fix sahlberg/libnfs@5b28ab5

@InuSasha: Hopefully with these updates (PR1612+PR1616) the jump to gcc-7.1 should be less painful, though maybe wait for gcc-7.2?

There's a gcc-6.3 bump available which might be another option.

If anyone wants to update gcc in a separate PR, feel free!

@lrusak

This comment has been minimized.

Show comment
Hide comment
@lrusak

lrusak May 10, 2017

Member

PS. all add-ons and binary add-ons should be bumped using the PKG_REV when merging this.

Member

lrusak commented May 10, 2017

PS. all add-ons and binary add-ons should be bumped using the PKG_REV when merging this.

@vpeter4

This comment has been minimized.

Show comment
Hide comment
@vpeter4

vpeter4 May 10, 2017

Contributor

Build was fine here too. Run-time test will follow.
Seems changes are fine.

Contributor

vpeter4 commented May 10, 2017

Build was fine here too. Run-time test will follow.
Seems changes are fine.

@MilhouseVH

This comment has been minimized.

Show comment
Hide comment
@MilhouseVH

MilhouseVH May 10, 2017

Contributor

PS. all add-ons and binary add-ons should be bumped using the PKG_REV when merging this.

Sounds like a script to do that would come in handy... Is it just a matter of iterating over addons and PKG_REV=$PKG_REV+1?

Contributor

MilhouseVH commented May 10, 2017

PS. all add-ons and binary add-ons should be bumped using the PKG_REV when merging this.

Sounds like a script to do that would come in handy... Is it just a matter of iterating over addons and PKG_REV=$PKG_REV+1?

@lrusak

This comment has been minimized.

Show comment
Hide comment
@lrusak

lrusak May 10, 2017

Member

@MilhouseVH tools/repo-tool will do this already for the regular add-ons, it should be adapted to include the binary add-ons also

./tools/repo-tool -ru
Member

lrusak commented May 10, 2017

@MilhouseVH tools/repo-tool will do this already for the regular add-ons, it should be adapted to include the binary add-ons also

./tools/repo-tool -ru
@MilhouseVH

This comment has been minimized.

Show comment
Hide comment
@MilhouseVH

MilhouseVH May 10, 2017

Contributor

Ok will take a look and add a commit to this PR

Contributor

MilhouseVH commented May 10, 2017

Ok will take a look and add a commit to this PR

@MilhouseVH

This comment has been minimized.

Show comment
Hide comment
@MilhouseVH

MilhouseVH May 10, 2017

Contributor

repo-tool updated to support official, binary, retroplayer and all option when bumping PKG_REV.

PKG_REV bumped for all addons (inc. retroplayer).

Contributor

MilhouseVH commented May 10, 2017

repo-tool updated to support official, binary, retroplayer and all option when bumping PKG_REV.

PKG_REV bumped for all addons (inc. retroplayer).

@MilhouseVH

This comment has been minimized.

Show comment
Hide comment
@MilhouseVH

MilhouseVH May 10, 2017

Contributor

I've removed Don't Merge and Wip labels, as this seems to be working fine now. I'll include this PR in the next testbuild.

Contributor

MilhouseVH commented May 10, 2017

I've removed Don't Merge and Wip labels, as this seems to be working fine now. I'll include this PR in the next testbuild.

@MilhouseVH

This comment has been minimized.

Show comment
Hide comment
@MilhouseVH

MilhouseVH May 11, 2017

Contributor

In relation to gcc-7.1, message from @stefansaraev (seo) on #irc:

[10-May-2017 19:51:55] <seo> ping Milhouse lrusak if you plan bumping to gcc 7.1. amlogic kernel is broken for me. ./drivers/amlogic/mmc/aml_sd_emmc.c booms. didn't have time to investigate further
[10-May-2017 19:52:24] <seo> cc @codesnake @peak3d ;)
[10-May-2017 19:54:03] <seo> Unhandled fault: alignment fault
Contributor

MilhouseVH commented May 11, 2017

In relation to gcc-7.1, message from @stefansaraev (seo) on #irc:

[10-May-2017 19:51:55] <seo> ping Milhouse lrusak if you plan bumping to gcc 7.1. amlogic kernel is broken for me. ./drivers/amlogic/mmc/aml_sd_emmc.c booms. didn't have time to investigate further
[10-May-2017 19:52:24] <seo> cc @codesnake @peak3d ;)
[10-May-2017 19:54:03] <seo> Unhandled fault: alignment fault
@lrusak

lrusak approved these changes May 12, 2017

@lrusak lrusak merged commit fbc5fb1 into LibreELEC:master May 12, 2017

MilhouseVH added a commit to MilhouseVH/LibreELEC.tv that referenced this pull request May 16, 2018

glibc: install localedef because the installer needs it
Looks like I cocked this up first with #1616 and then #1686.

MilhouseVH added a commit to MilhouseVH/LibreELEC.tv that referenced this pull request May 16, 2018

glibc: install localedef because the installer needs it
Looks like I cocked this up first with #1616 and then #1686.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment