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

libglog: Switch to nanosleep #9612

Merged
merged 2 commits into from
Aug 2, 2019
Merged

libglog: Switch to nanosleep #9612

merged 2 commits into from
Aug 2, 2019

Conversation

neheb
Copy link
Contributor

@neheb neheb commented Jul 31, 2019

Optionally fixes compilation with uClibc-ng.

Also switched to CMake. Allows removal of InstallDev as well as further
simplification. Also 100KB smaller package.

Added some C/LDFLAGS to further reduce the size.

Signed-off-by: Rosen Penev rosenp@gmail.com

Maintainer: @amir-sabbaghi
Compile tested: arc700

@karlp
Copy link
Contributor

karlp commented Jul 31, 2019

your title is misleading, these are very separate things to be doing to the package, all tryign to sneak in under "use nanosleep" File these separately, or at least as separate commits. I sitll have same comments as I'm going to have on all of these, whhere's the upstream ticket for the usleep/nanosleep conversion?

@neheb
Copy link
Contributor Author

neheb commented Jul 31, 2019

google/glog#460

Copy link
Contributor

@amir-sabbaghi amir-sabbaghi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would also appreciate it if you make separate PRs for usleep and cmake.

libs/libglog/Makefile Outdated Show resolved Hide resolved
libs/libglog/patches/110-nanosleep.patch Show resolved Hide resolved
Allows removal of InstallDev as well as further simplification. Also 100KB
smaller package.

Added some C/LDFLAGS to further reduce the size.

Small Makefile cleanups.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
Optionally fixes compilation with uClibc-ng.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
@neheb
Copy link
Contributor Author

neheb commented Jul 31, 2019

Should be good now.

@amir-sabbaghi
Copy link
Contributor

Well you didn't create a separate PR. But it looks good to me anyway.

@diizzyy
Copy link
Contributor

diizzyy commented Aug 1, 2019

Just wait on upstream, it just adds unnecessary overhead

@neheb
Copy link
Contributor Author

neheb commented Aug 2, 2019

@amir-sabbaghi Well, the main benefit of separate PRs is running through both commits through CircleCI individually. Both do in this case as the second commit is only for uClibc-ng.

As for upstream, it seems they're relatively inactive. The other patch for glog was sent two months ago. No comment. google/glog#443

@neheb neheb merged commit 54f89ea into openwrt:master Aug 2, 2019
@neheb neheb deleted the glog branch August 2, 2019 00:17
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