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

Make release scripts architecture-agnostic and add ARM support #25591

Merged
merged 1 commit into from
Aug 11, 2016

Conversation

tiborvass
Copy link
Contributor

This patch allows to only release the packages that were built and are
present under the bundles/ directory instead of assuming packages exist
for all distros enumerated in the contrib/builder/ directory.

It also now adds support for armhf architecture for apt repositories.

Signed-off-by: Tibor Vass [email protected]

@tiborvass
Copy link
Contributor Author

Ping @tianon @mlaventure

@tianon
Copy link
Member

tianon commented Aug 10, 2016

Nice, LGTM

@mlaventure
Copy link
Contributor

LGTM

@tiborvass tiborvass closed this Aug 11, 2016
@thaJeztah
Copy link
Member

why was this closed? didn't work?

@tiborvass
Copy link
Contributor Author

Yes, I have a fix but need to verify something else and didn't want this merged.

@tiborvass
Copy link
Contributor Author

Ping @tianon again, sorry. For release-rpm this time.

@tianon
Copy link
Member

tianon commented Aug 11, 2016

Nice, updated content still LGTM 👍

@mlaventure
Copy link
Contributor

distros=( fedora centos opensuse oraclelinux ) is not used anymore, you can delete that line (and the comment above)

This patch allows to only release the packages that were built and are
present under the bundles/ directory instead of assuming packages exist
for all distros enumerated in the contrib/builder/ directory.

It also now adds support for armhf architecture for apt repositories.

Signed-off-by: Tibor Vass <[email protected]>
@tiborvass
Copy link
Contributor Author

@mlaventure fixed, thanks!

@mlaventure
Copy link
Contributor

LGTM!

@DieterReuter
Copy link
Contributor

@tiborvass really nice patch. I already tried to resolve this without success so far, but your solution looks pretty good!

@tiborvass tiborvass deleted the fix-release-script branch July 17, 2019 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants