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

Move to PowerDNS Recursor 5.2.0-rc1 (for build test) #71

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

omoerbeek
Copy link
Member

Maintainer: me / @<github-user> (find it by checking history of the package Makefile)
Compile tested: (put here arch, model, OpenWrt version)
Run tested: (put here arch, model, OpenWrt version, tests done)

Description:

@omoerbeek
Copy link
Member Author

Something is not quite right still...

@Habbie
Copy link
Member

Habbie commented Dec 16, 2024

needed a rebase, you worked on a branch that still had 5.1.2 while upstream had 5.1.3 already

@Habbie
Copy link
Member

Habbie commented Dec 16, 2024

(and it is possible I caused this by updating master on our fork when I added the refresh commit)

@omoerbeek
Copy link
Member Author

omoerbeek commented Dec 16, 2024

Build succeeded (at least the one I checked), it just does not like my version name.

@Habbie
Copy link
Member

Habbie commented Dec 17, 2024

ah, openwrt#25488

@Habbie
Copy link
Member

Habbie commented Dec 17, 2024

in the Alpine (which uses APK, but not openwrt-like Makefiles) APKBUILD I have builddir="$srcdir/$pkgname-${pkgver//_/-}" and specify 5.2.0_rc1 for these tests. Need to check if we can do something similar for openwrt. Alternatively, we'd just override the tarball file name for this test.

@omoerbeek
Copy link
Member Author

I'll try underscore soon.

@omoerbeek
Copy link
Member Author

Grumble, that did not work. The build expect the extracted tarball to have path names corresponding to the package name it seems.

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.

2 participants