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

New package: grandorgue-3.13.0 #46711

Closed
wants to merge 1 commit into from

Conversation

nicolodavis
Copy link

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, (x86_64 glibc)

@nicolodavis
Copy link
Author

Just curious if I need to do anything at my end to get the CI to green tick this PR?

Copy link

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

@github-actions github-actions bot added the Stale label Jan 17, 2024
@nicolodavis
Copy link
Author

Hi @classabbyamp, is Void still accepting new package contributions? Let me know if I should close this PR instead.

@classabbyamp
Copy link
Member

we do accept new package contributions, but generally it requires a maintainer to be either interested in the package or see the utility of merging it

we also generally like to see new contributors make some contributions (updates, fixes, etc) to existing packages (especially orphaned packages) before adding new packages.

revision=1
archs="x86_64"
build_style="cmake"
configure_args="-DCMAKE_BUILD_TYPE=Release -G Ninja"
Copy link
Member

Choose a reason for hiding this comment

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

neither of these need to be set. void sets the build type to None and the generator to ninja by default in common/build-style/cmake.sh

export WX_CONFIG=wx-config-gtk3

do_fetch() {
git clone --recurse-submodules --depth 1 --branch ${version}-${revision} https://github.com/GrandOrgue/grandorgue.git ${wrksrc}
Copy link
Member

Choose a reason for hiding this comment

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

do not use git clone to fetch. if a packages has submodules, they should be added as distfiles. see the edk2-ovmf template for an example.

license="GPL-2.0-or-later"
homepage="https://github.com/GrandOrgue/grandorgue"

export WX_CONFIG=wx-config-gtk3
Copy link
Member

Choose a reason for hiding this comment

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

this can probably be replaced with build_helper="cmake-wxWidgets-gtk3"

@github-actions github-actions bot removed the Stale label Jan 18, 2024
Copy link

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

@github-actions github-actions bot added the Stale label Apr 18, 2024
@github-actions github-actions bot closed this May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants