meetalva / alva

Create living prototypes with code components.
https://meetalva.io
MIT License
3.74k stars 226 forks source link

Alternative installation instruction: AUR #638

Closed gunar closed 5 years ago

gunar commented 5 years ago

With alva-git in place, installing Alva into Arch/Manjaro systems is now just a matter of executing yaourt -Syy alva-git.

rsteube commented 5 years ago
diff --git a/.SRCINFO b/.SRCINFO
index 5977274..13f5817 100644
--- a/.SRCINFO
+++ b/.SRCINFO
@@ -1,16 +1,23 @@
 pkgbase = alva-git
-   pkgdesc = Create living prototypes with code components. 
-   pkgver = 1
+   pkgdesc = Create living prototypes with code components.
+   pkgver = v0.8.0
    pkgrel = 1
    url = https://meetalva.io
    arch = i686
    arch = x86_64
    license = MIT
    makedepends = git
-   depends = glibc
+   makedepends = npm
+   makedepends = nodejs
+   depends = gconf
+   depends = libxss
+   depends = nss
+   depends = gtk3
+   depends = electron
+   depends = libicns
    provides = alva
    conflicts = alva
-   source = reredirect-git::git+http://github.com/jerome-pouiller/reredirect.git
+   source = alva-git::git+http://github.com/meetalva/alva.git
    md5sums = SKIP

 pkgname = alva-git
diff --git a/PKGBUILD b/PKGBUILD
index cc7d61b..f97c7b4 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -26,16 +26,16 @@ md5sums=(SKIP)
 pkgver() {
   cd "$srcdir/$pkgname"
 # Use latest tag
-  git tags|tail -n 1
+  git tag|tail -n 1
 }

 build() {
   cd "$pkgname"
-  last_tag=$( git tags|tail -n 1 )
+  last_tag=$( git tag|tail -n 1 )
   git checkout $last_tag
   npm install
   npm run build
-  npm run build:electron || true
+  node_modules/.bin/electron-builder -l dir
 }

 package() {
KnisterPeter commented 5 years ago

Unfortunately it fails with the message npm required.

gunar commented 5 years ago

@rsteube thanks!

  1. .SRCINFO updated, thanks!
  2. Nice catch
  3. Agreed, implemented
rsteube commented 5 years ago

missed one tags at https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=alva-git#n29 and the dir parameter for electron-builder (this one is not necessary, but skips Appimage and Snap)

builds and installs on my machine with tags fixed

gunar commented 5 years ago

@rsteube done! I'm still getting this error though, do you think it's safe to just ignore it?

image

rsteube commented 5 years ago

yes, that should be fixed with #641 - the dependencies are currently declared as devDependencies and thus missing at runtime. i also had the electron drag&drop bug before and had to start it with --disable-gpu, but last time i started it i didn't need it

rsteube commented 5 years ago

ok, --disable-gpu is still necessary for me (might as well add that to the script until the bug is fixed) @marionebl i can confirm it installs successfully on my machine

marionebl commented 5 years ago

@gunar: We'd like to co-maintain the alva-git AUR and mirror it here on GitHub. You might also be interested in alva-appimage :)

Would you be interested in making me a co-maintainer and get admin permissions on the GitHub mirror in return?

gunar commented 5 years ago

@marionebl sure! what's your AUR username?

marionebl commented 5 years ago

@gunar I am https://aur.archlinux.org/account/marionebl

gunar commented 5 years ago

Woohoo! image

marionebl commented 5 years ago

I added you as collaborator to https://github.com/meetalva/aur-alva-git @gunar. We'll have a look at setting up TravisCI to check the build script and push changes to AUR.

marionebl commented 5 years ago

Closing in favor of #166