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

routinator: update to 0.11.2 #35971

Merged
merged 1 commit into from
Jul 3, 2022
Merged

Conversation

klarasm
Copy link
Contributor

@klarasm klarasm commented Mar 4, 2022

in addition:

  • upstream has removed doc/misc.md
  • add changelog

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture, (x86_64-musl)
  • I built this PR locally for these architectures (crossbuilds last on 0.11.1):
    • aarch64-musl
    • armv7l
    • armv6l-musl

@klarasm klarasm force-pushed the routinator-update branch 2 times, most recently from 7e37ee2 to 9fccd74 Compare April 10, 2022 15:08
@klarasm klarasm changed the title routinator: update to 0.11.0 routinator: update to 0.11.1 Apr 10, 2022
@klarasm
Copy link
Contributor Author

klarasm commented Apr 10, 2022

Update pull request as upstream has released 0.11.1

@klarasm klarasm changed the title routinator: update to 0.11.1 routinator: update to 0.11.2 Apr 24, 2022
@klarasm
Copy link
Contributor Author

klarasm commented Apr 24, 2022

Update pull request as upstream has release 0.11.2. Did not redo local crossbuilds due to minimal changes from 0.11.1.

revision=1
build_style=cargo
depends="rsync"
short_desc="Resource Public Key Infrastructure (RPKI) validator"
maintainer="Cameron Nemo <cnemo@tutanota.com>"
license="BSD-3-Clause"
homepage="https://rpki.readthedocs.io/"
changelog="https://raw.githubusercontent.com/NLnetLabs/routinator/v${version}/Changelog.md"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
changelog="https://raw.githubusercontent.com/NLnetLabs/routinator/v${version}/Changelog.md"
changelog="https://raw.githubusercontent.com/NLnetLabs/routinator/main/Changelog.md"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@paper42
Copy link
Member

paper42 commented Apr 25, 2022

@CameronNemo does this sound ok?

@klarasm
Copy link
Contributor Author

klarasm commented Apr 25, 2022

Seems like github packages is having issues (https://www.githubstatus.com/incidents/b40k7ckrs7sp) which is probably why some builds are failing.

Starting job container
  /usr/bin/docker --config /home/runner/work/_temp/.docker_500e6f68-fa9e-4be4-a631-8fb3451b5bfe login ghcr.io -u klarasm --password-stdin
  /usr/bin/docker --config /home/runner/work/_temp/.docker_500e6f68-fa9e-4be4-a631-8fb3451b5bfe pull ghcr.io/void-linux/xbps-src-masterdir:20210313rc01-x86_64-musl
  Error response from daemon: received unexpected HTTP status: 500 Internal Server Error

in addition:
- upstream has removed doc/misc.md
- add changelog
@CameronNemo
Copy link
Contributor

@paper42 Yeah this looks fine.

@klarasm
Copy link
Contributor Author

klarasm commented Jul 3, 2022

Is there anything else that's needed for this to be merged?

@paper42 paper42 merged commit 3fb0044 into void-linux:master Jul 3, 2022
@CameronNemo
Copy link
Contributor

@klarasm if you want to do anything else to better integrate routinator into the distro:

  • write a runit service
  • build and install the user manual (doc/manual)

@klarasm
Copy link
Contributor Author

klarasm commented Jul 4, 2022

Will take a look. I already have a runit service I use but will probably have to modify it a little for it to be suitable generally. I'll create a draft PR once I have something.

@klarasm klarasm deleted the routinator-update branch September 28, 2022 21:31
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.

None yet

3 participants