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

PKGBUILD: update name to tiny-git, sync other changes #424

Merged
merged 5 commits into from
Jan 5, 2024

Conversation

ralphptorres
Copy link
Contributor

Changes include:

  • refactor and add functions
  • disable link-time optimization (due to compilation issues in some machines)
  • rename as tiny-git and update info

all: prepend cd with srcdir var
pkgver:
    simplify sed command
    abbreviate commit id
prepare: added, fetch needed libraries in advance
build:
    do cargo build instead of cargo install
    change options
check: added, run prescribed tests
package: use -t option
In some machines, building the package fails due to ring crate and (gcc)
compiler linking issue.

Refer: briansmith/ring#1444
@osa1
Copy link
Owner

osa1 commented Jan 1, 2024

Does this file have to live in this repo? It's probably easier to update the AUR packages than this repo (I don't have as much time anymore), so I suggest we update the links (as in this PR) but remove the file. That way there will be just one copy of the PKGBUILD.

@ralphptorres
Copy link
Contributor Author

I understand, would be fine with that.

@osa1
Copy link
Owner

osa1 commented Jan 3, 2024

Thanks. The README change in this PR are good, if you could revert the pkgbuild, or even better, delete it, we can merge this.

@ralphptorres
Copy link
Contributor Author

Done

@ralphptorres
Copy link
Contributor Author

Suggest updating commit message for merge. Thanks

@osa1 osa1 merged commit 41596ab into osa1:master Jan 5, 2024
7 checks passed
@osa1
Copy link
Owner

osa1 commented Jan 5, 2024

Thanks!

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.

2 participants