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

pkg: danctnix: osk-sdl: upgrade to 0.67 #463

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ShapeShifter499
Copy link
Contributor

No description provided.

@ShapeShifter499 ShapeShifter499 changed the title Provide fixes and upgrade osk-sdl to 0.67 pkg: danctnix: osk-sdl: upgrade to 0.67 Sep 18, 2022
@Danct12
Copy link
Member

Danct12 commented Sep 18, 2022

Please squash all the commits and provide details about the modification you made in commit message.

@ShapeShifter499
Copy link
Contributor Author

How does this look?

@ShapeShifter499
Copy link
Contributor Author

made a couple of minor edits, should be good now

@Danct12
Copy link
Member

Danct12 commented Sep 19, 2022

Can you switch the package to VCS and put the commit in a variable? Such example is here and VCS package guidelines.

Other than that, LGTM.

@ShapeShifter499
Copy link
Contributor Author

ShapeShifter499 commented Sep 19, 2022

There, made the update

EDIT: hang on

@ShapeShifter499
Copy link
Contributor Author

Okay now it should be good, forgot that I had to also update where it would 'cd' into for the source

@Danct12
Copy link
Member

Danct12 commented Sep 20, 2022

Since this is a package pulled from git, please use pkgver function. See VCS package guidelines

@ShapeShifter499
Copy link
Contributor Author

Okay well should I tie this package to the latest source like how a "-git" package works on the AUR or just keep the commit tag but include the git tag as the pkgver?

*Remove unneeded quiet line. It doesn't do anything in our hook

*Ensure IFS is reset. Not resetting IFS may conflict with other hooks and kernel command line variables

*Change PKGBUILD to build 0.67
@ShapeShifter499
Copy link
Contributor Author

Like this?

@ShapeShifter499
Copy link
Contributor Author

Any updates?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants