Package Details: fe 1.2.0-1

Git Clone URL: https://aur.archlinux.org/fe.git (read-only, click to copy)
Package Base: fe
Description: AUR helper with a familiar subcommand system
Upstream URL: https://github.com/baris-inandi/fe
Licenses: GPL3
Submitter: baris-inandi
Maintainer: baris-inandi
Last Packager: baris-inandi
Votes: 2
Popularity: 0.75
First Submitted: 2022-07-31 21:49 (UTC)
Last Updated: 2022-08-15 19:40 (UTC)

Latest Comments

« First ‹ Previous 1 2 3 Next › Last »

yochananmarqos commented on 2022-08-09 18:31 (UTC)

@baris-inandi: I think you meant to bump the pkgver from 1.1.5 to 1.1.6? 0 is not a valid pkgver.

baris-inandi commented on 2022-08-09 18:30 (UTC)

@yochananmarqos

I read the vcs package guidelines and came up with this pkgver():

pkgver=0
...
pkgver() {
    cd "$pkgname"
    printf "%s" "$(git describe --tags --long)"
}

However, just adding this function does not update the version. I expected makepkg --printsrcinfo > .SRCINFO to include the latest tag but it shows the version as 0.

yochananmarqos commented on 2022-08-08 22:59 (UTC)

@baris-inandi: It appears you missed my edit to my previous comment.

yochananmarqos commented on 2022-08-01 18:26 (UTC) (edited on 2022-08-01 18:28 (UTC) by yochananmarqos)

@a821: Aha! Would you believe I never noticed that? The request is cancelled now.

@baris-inandi: You can use a pkgver() function, see VCS package guidelines.

baris-inandi commented on 2022-08-01 18:00 (UTC) (edited on 2022-08-01 18:04 (UTC) by baris-inandi)

Hey, just implemented git tags using git checkout. I also removed the maintainer variable and replaced it with a comment. (1.0.19)

a821 commented on 2022-08-01 17:36 (UTC)

@baris-inandi: As said, the PKGBUILD is much better. Just minor issues

  1. you can easily pin a commit (while you work on the tags) with for example source=("git+$url#commit=ad5db3d79c4ca66e28b79cdd801e088c8c946b52")

  2. the maintainer line should be a bash comment, not a variable, eg, # Maintainer: your name <your email> (custom variables in the PKGBUILD need an underscore prefix: _foobar="some value")

@yochananmarqos: btw, you can reject the deletion request yourself ("Requests" tab) to reduce the load of the TUs :)

yochananmarqos commented on 2022-08-01 14:46 (UTC)

@baris-inandi: It's much better now, thanks for working on it. Perhaps I was a little hasty filing the deletion request. The request will be rejected since you resolved the issue.

baris-inandi commented on 2022-08-01 13:22 (UTC) (edited on 2022-08-01 13:58 (UTC) by baris-inandi)

Hey @a821, thank you so much for the help, really appreciate it.

Here's what I did (1.0.14):
- Removed all sudo calls
- Added paru to depends
- Removed bash from makedepends
- Added the config files to backup
- Changed license to "GPL3"
- Now the PKGBUILD uses install properly

I'll try to use git-tags too so that the naming is also appropriate

a821 commented on 2022-08-01 11:43 (UTC)

The PKGBUILD is extremely bad. Please read carefully the Go package guidelines again (link in first comment). There's even a sample PKGBUILD you can mostly copy and paste.

  1. This package uses git sources, so it needs the -git suffix, unless you pin a commit or use git-tags.
  2. You do not need sudo at all.
  3. Use the install command to install files to $pkgdir, for example 'install -Dm755 my_program "$pkgdir/usr/bin/my_program"`
  4. If you intend to install paru.conf to /etc then your package will conflict with paru which, I guess, is not what you want.
  5. If your package needs paru, add it to the depends array.
  6. If your package installs files that could be modified by the user (eg, files in /etc), then add those files to the backup array.
  7. bash is not required in makedepends as it is provided by base-devel (which it is assumed to be installed) via pacman
  8. license should be simply GPL3 (no "v")

baris-inandi commented on 2022-07-31 22:38 (UTC) (edited on 2022-07-31 22:55 (UTC) by baris-inandi)

Hello, does the PKGBUILD look right now? I'm still calling sudo for go build on the build() function, should sudo be avoided entirely?