Package Details: gotop-git 3.5.1.r4.g84ec9e4-1

Git Clone URL: https://aur.archlinux.org/gotop-git.git (read-only, click to copy)
Package Base: gotop-git
Description: A terminal based graphical activity monitor inspired by gtop and vtop
Upstream URL: https://github.com/xxxserxxx/gotop
Licenses: AGPL3
Conflicts: gotop
Provides: gotop
Submitter: m3thodic
Maintainer: m3thodic
Last Packager: m3thodic
Votes: 3
Popularity: 0.024844
First Submitted: 2018-11-13 10:52
Last Updated: 2020-04-21 13:52

Dependencies (2)

Required by (0)

Sources (1)

Latest Comments

1 2 Next › Last »

caleb commented on 2020-03-06 20:35

The pkgver value should not have the "v" in it, that needs to be striped out in the pkgver() function.

m3thodic commented on 2020-03-06 18:05

Upstream url has been updated, thanks.

caleb commented on 2020-02-28 04:51

Please don't "version bump" -git or other VCS based packages! Only update when there actual changes to the build process other than the version which will be regenerated at build time anyway.

caleb commented on 2020-02-24 04:47

Second the call to change the upstream to follow serxxx's fork. See discussion on gotop and nate gotop-bin is also moved over too.

FabioLolix commented on 2020-02-23 22:31

Updated pkgbuild: https://github.com/FabioLolix/PKGBUILD-AUR_fix/blob/master/g/gotop-git/PKGBUILD

tannicanteater commented on 2020-02-23 22:24

Hi, would it be possible to change the source of this package to the fork maintained by @serxxx ?

https://github.com/xxxserxxx/gotop

This issue explains the recent developments with gotop:

https://github.com/cjbassi/gotop/issues/187

Also, the other two gotop packages have already been changed to point to serxxx's fork, if that has any bearing on your decision.

anodium commented on 2019-07-24 00:32

Hi @m3thodic,

I've added a desktop entry and icon so that it can be run from a DE directly. Here's the patch, you can apply it with git am 1-gotop-git.patch.

I've also updated the git revision in a seperate patch.

cjbassi commented on 2019-02-25 21:37

@egrupled Thanks again. Should be fixed now. I applied the same changes that I described in the gotop AUR package.

egrupled commented on 2019-02-25 20:07

There are couple issues with the current PKGBUILD:

  1. It creates persistent ~/go dir (with default GOPATH) and download all dependencies there. Moreover files in that dir are set as read-only.

  2. It doesn't honor LDFLAGS

Below is example which fixes this issue:

  1. It sets GOPATH to temporary directory under build dir.

  2. It take LDFLAGS into account, also strips build path for reproducibility[1].

  3. It makes temporary GOPATH writable which allows for clean up.

[1] https://wiki.archlinux.org/index.php/Go_package_guidelines#Flags_and_build_options

Please consider adopting those changes.

build() {
  cd "$srcdir/$pkgname"
  export GOPATH="$srcdir/gopath"
  go build \
  -gcflags "all=-trimpath=${PWD}" \
  -asmflags "all=-trimpath=${PWD}" \
  -ldflags "-extldflags ${LDFLAGS}"
}

package() {
  export GOPATH="$srcdir/gopath"
  install -Dm755 "$srcdir/$pkgname/gotop" "$pkgdir/usr/bin/gotop"
  chmod -R 644 "$GOPATH"
}

m3thodic commented on 2019-02-21 01:13

@egrupled thanks for the heads up, PKGBUILD has been updated.