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

ntpd compilation fails #25266

Closed
wwortel opened this issue Nov 7, 2024 · 6 comments
Closed

ntpd compilation fails #25266

wwortel opened this issue Nov 7, 2024 · 6 comments

Comments

@wwortel
Copy link

wwortel commented Nov 7, 2024

ntpd
I need ntpd in my images but compile of any OpenWrt image fails at ntpd.
Have also tried p18 which is at ntpd' source but same problem. 'make V=s' shows:
[ ! -d ./src/ ] || cp -fpR ./src/. /var/sdks/OpenWrt/master/build_dir/target-mipsel_24kc_musl/ntp-4.2.8p17

Applying /var/sdks/feeds/packages/net/ntpd/patches/0001-libntp-Do-not-use-PTHREAD_STACK_MIN-on-glibc.patch using plaintext:
patching file libntp/work_thread.c
Hunk #1 FAILED at 41.
1 out of 1 hunk FAILED -- saving rejects to file libntp/work_thread.c.rej
Patch failed! Please fix /var/sdks/feeds/packages/net/ntpd/patches/0001-libntp-Do-not-use-PTHREAD_STACK_MIN-on-glibc.patch!
work_thread.c.rej.txt

@brada4
Copy link

brada4 commented Nov 10, 2024

Please fill the issue template incl diffconfig.

@brada4
Copy link

brada4 commented Nov 10, 2024

particular patch applies to ntpd .... p15 and EOL-d v22. If you are experimenting in archaeology it is up to you to backport (in this case delete) patches.
Or use stable version.

@wwortel
Copy link
Author

wwortel commented Nov 10, 2024

Mmm, experimenting archeology?? After getting stuck I actually did a completely fresh install of OpenWrt, and just maintained links to 'feeds' and 'dl' folders, but updated all packages using './scripts/feeds update -a'
Have not touched ntpd at all, except its make file but only after having stumbled over this compile error to test the most recent v18 version, with same problem. So, judging from your comment, the packages update seems not to have produced a coherent situation between patches and ntpd source code. Looking in this repository /net/ntpd I actually do not see any patches folder.
Should I just remove it in my local copy of ntpd?

@brada4
Copy link

brada4 commented Nov 10, 2024

The patch is only present in ntpd for OpenWRT 22 branch for ntpd .... p15, apparently included in 23 p17 and no longer needed.
Something is mismatched between core sources and package tree if you are trying to build legit 23/24/snapshot images.

@wwortel
Copy link
Author

wwortel commented Nov 10, 2024

Have removed 'feeds' completely and reinstalled it, and problem gone. Thanks you for pointing out the the branches issue. Not sure why in the past the 'feeds' got out of sync with the rest doing updates. When doing a git pull of OpenWrt I always also run the script to update package feeds. Anyway, issue resolved, I'll close the issue.

@wwortel wwortel closed this as completed Nov 10, 2024
@brada4
Copy link

brada4 commented Nov 10, 2024

You can click release branches in github and literarily compare file listings and git log to ones on disk for breaking package. Also check faillogs on download server if no problem pops up in first check.

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

No branches or pull requests

2 participants