cloudhead / rx

👾 Modern and minimalist pixel editor
https://discord.gg/xHggPjfsS9
GNU General Public License v3.0
3.1k stars 109 forks source link

Not building on latest commit #39

Closed parkerlreed closed 4 years ago

parkerlreed commented 4 years ago

Vulkan is setup and working on the system.

Arch Linux https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=rx-git vulkan-intel 19.2.2-1 vulkan-headers 1:1.1.126-1

==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> Retrieving sources...
  -> Updating rx git repo...
Fetching origin
==> Validating source files with sha512sums...
    rx ... Skipped
==> Removing existing $srcdir/ directory...
==> Extracting sources...
  -> Creating working copy of rx git repo...
Cloning into 'rx'...
done.
==> Starting pkgver()...
==> Updated version: rx-git 0.2.0+r108+ga2446d0-1
==> Sources are ready.
==> Making package: rx-git 0.2.0+r108+ga2446d0-1 (Mon 04 Nov 2019 06:04:59 PM EST)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> WARNING: Using existing $srcdir/ tree
==> Starting pkgver()...
==> Removing existing $pkgdir/ directory...
==> Starting build()...
error: Package `rx v0.2.0 (/home/parker/.cache/yay/rx-git/src/rx)` does not have these features: `vulkan`
==> ERROR: A failure occurred in build().
    Aborting...
Error making: rx-git
parkerlreed commented 4 years ago

So according to this the vulkan option was removed. https://github.com/cloudhead/rx/commit/bcb13adb359a6dd478341ca527fa9a4881121c96

The Readme needs to be updated to reflect whatever the current build instructions are

https://github.com/cloudhead/rx/blob/master/README#L83

cloudhead commented 4 years ago

Good point, will update.

parkerlreed commented 4 years ago

Thanks. I will mention this on the AUR comment page.

Compiled and works.