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: Tuner-1.5.1 #41112

Closed
wants to merge 1 commit into from
Closed

New package: Tuner-1.5.1 #41112

wants to merge 1 commit into from

Conversation

MechDR
Copy link

@MechDR MechDR commented Dec 15, 2022

Testing the changes

  • I tested the changes in this PR: YES

@MechDR
Copy link
Author

MechDR commented Dec 16, 2022

Any suggestions on how to fix these issues?

@classabbyamp classabbyamp added the new-package This PR adds a new package label Dec 16, 2022
@oreo639
Copy link
Member

oreo639 commented Dec 17, 2022

glib needs to be in hostmakedepends (as well as makedepends).
hostmakedepends is for packages that will be installed for the build-machine's architecture and makedepends is for packages that will be installed for the target's architecture. Those will be the same when native compiling and different when cross-compiling. (in the case of glib, you need both)

Also subject does not follow CONTRIBUTING.md guildelines, your commit message doesn't follow the guidelines. (it should be New package: Tuner-1.5.1)

You can fix the commit message using git commit --amend

https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md

@MechDR MechDR changed the title Tuner template (final) New package: Tuner-1.5.1 Dec 20, 2022
@MechDR
Copy link
Author

MechDR commented Dec 20, 2022

@oreo639 Is this OK?

@MechDR
Copy link
Author

MechDR commented Dec 20, 2022

OK, everything failed, what's wrong now...

@paper42
Copy link
Member

paper42 commented Dec 20, 2022

OK, everything failed, what's wrong now...

exactly what it's telling you

Your branches is based on too old copy.
Please rebase to newest copy.

@paper42
Copy link
Member

paper42 commented Dec 20, 2022

also, this PR should contain one commit, squash them please

@MechDR
Copy link
Author

MechDR commented Dec 20, 2022

Let's see...

@MechDR
Copy link
Author

MechDR commented Dec 21, 2022

I synced my fork with master, it still reports the same error...

@Chocimier
Copy link
Member

glib-compile-resources is in glib-devel so you need to add that to hostmakedepends.

You need to rebase and force push the submitted branch, not master.

@oreo639
Copy link
Member

oreo639 commented Dec 21, 2022

Also, I recommend you use the command line. There are a lot of thing you cannot do from the web ui.

I recommend you read:
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md

You can fix the commit history with the following:

git clone https://github.com/MechDR/void-packages/ -b tuner
cd void-packages
git remote add upstream https://github.com/void-linux/void-packages.git
git reset --soft HEAD~2
git commit --amend
git push -f
git pull --rebase upstream master
git push -f

The commit needs to be named New package: Tuner-1.5.1.

In the future, do git add <file>, git commit --amend, git push -f

@MechDR
Copy link
Author

MechDR commented Dec 23, 2022

@oreo639 I do appreciate the effort and the mini command tutorial you did, but I'm stuck...

git pull --rebase upstream master

error: cannot pull with rebase: You have unstaged changes.
error: please commit or stash them.

Now what...

@paper42
Copy link
Member

paper42 commented Dec 23, 2022

You have changes that you didn't commit, run git diff or git status to see them and do something about them - either remove them with git restore, put them away for a while with git stash or commit them with git add and git commit

revision=1
build_style=meson
hostmakedepends="glib gettext pkg-config vala"
makedepends="glib geoclue2-devel geocode-glib-libsoup2-devel glib-devel
Copy link
Member

Choose a reason for hiding this comment

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

I am against including a new package that uses libsoup2, especially geocode-glib-libsoup2, we should be trying to slowly get rid of those packages.

Copy link
Author

@MechDR MechDR Jan 12, 2023

Choose a reason for hiding this comment

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

Things should be resolved in next versions, as the dev seems to be on board about dropping support for user location all together in the new milestone version 1.6.

louis77/tuner#106

So, this would only be a temporary solution, just to get Tuner in the src collection.

Sorry for the delay, I don't have a lot of free time currently, I'll try and get this done next week or so.

PS: I'm sorry if I sounded kinda harsh with my previous comments, it's personal real life problems, nothing to do with anyone I replied to, people have been more than helpful here.

@github-actions
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.

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

6 participants