Package Details: saber 0.21.3-1

Git Clone URL: https://aur.archlinux.org/saber.git (read-only, click to copy)
Package Base: saber
Description: The cross-platform notes app built for handwriting
Upstream URL: https://github.com/saber-notes/saber
Licenses: GPL-3.0-or-later
Submitter: PolarianDev
Maintainer: mhdi
Last Packager: mhdi
Votes: 4
Popularity: 0.006657
First Submitted: 2023-06-02 11:25 (UTC)
Last Updated: 2024-04-01 22:45 (UTC)

Latest Comments

1 2 Next › Last »

mhdi commented on 2024-04-02 10:27 (UTC) (edited on 2024-04-02 10:28 (UTC) by mhdi)

@PolarianDev No worries at all. :)

Yeah, there was a typo and I somehow missed it.

Thanks

PolarianDev commented on 2024-04-01 23:19 (UTC)

Ah, you fixed this 34 mins ago. Nevermind :)

PolarianDev commented on 2024-04-01 23:18 (UTC)

@mhdi Apologies, I was not aware of this, I have always just used /bin/bash and didn't pay much attention.

The actual issue here is a typo you wrote /usb/bin/bash, please change this to /usr/bin/bash, otherwise saber can't launch :P

Apologies for the mistake,
Polarian

mhdi commented on 2024-03-25 10:42 (UTC)

@PolarianDev Thank you for your comment. In Arch Linux, /bin is a symbolic link to /usr/bin:

$ realpath /bin
/usr/bin

Moreover, all the executables of the bash package are located within /usr/bin and there are none within /bin:

$ pacman -Fl bash
...
bash usr/bin/
bash usr/bin/bash
bash usr/bin/bashbug
bash usr/bin/rbash
bash usr/bin/sh
...

Thus, the correct shebang is /usr/bin/bash.

PolarianDev commented on 2024-03-25 00:29 (UTC)

Hello,

Firstly, fvm seems to be a much better way of managing the horrific flutter issues, I am going to suggest this to a staff member and see their thoughts on it.

However your change in the shebang from /bin/bash to /usr/bin/bash is incorrect, /usr/bin/bash does not exist on Arch Linux systems, it is /bin/bash, this causes the wrapper script to fail.

Please revert that change.

Thanks,
Polarian

PolarianDev commented on 2023-08-24 22:58 (UTC)

Sorry for the late update.

I managed to build the latest version with the current version of flutter.

Like I said, this package is unstable, but currently its up to date now :D

Have a good night!

Polarian

PolarianDev commented on 2023-06-09 12:16 (UTC) (edited on 2023-06-09 12:17 (UTC) by PolarianDev)

@LuciRose431 After discussing this on IRC, I have settled on the choice that I will not change this, and I will explain this now:

  • bash is within every arch installation, it cannot be uninstalled therefore bash always exists.
  • The script works fine with bash, using /bin/sh could mean a shell does not support the features etc, although this is a wrapper script which every shell should support, as a rule of thumb its just better to keep it to whats been tested.

There is little to no reason to execute the script under a different shell, all the script does is changes the loader path to the flutter libraries and then executes the native linux flutter application.

Therefore I am going to keep the script how it is, sorry...

LuciRose431 commented on 2023-06-09 09:41 (UTC) (edited on 2023-06-09 09:41 (UTC) by LuciRose431)

How about change #!/bin/bash -> #!/bin/sh?

PolarianDev commented on 2023-06-06 10:08 (UTC)

@LuciRose431 it is a dependency yes, saber.sh requires bash, the reason there is a wrapper script for saber instead of executing the binary directly is due to the library path being different, executing saber directory causes a loader error, please see the flutter packaging guideline draft which explains this:

https://wiki.archlinux.org/title/User:Gromit/Flutter_package_guidelines#Loader_fails_to_find_shared_object_(.so)_files

LuciRose431 commented on 2023-06-06 02:32 (UTC)

is bash really dependency here? Look at this

1 2 Next › Last »