Skip to content

"AM" 10.1.1#2318

Open
ivan-hc wants to merge 22 commits intomainfrom
dev
Open

"AM" 10.1.1#2318
ivan-hc wants to merge 22 commits intomainfrom
dev

Conversation

@ivan-hc
Copy link
Copy Markdown
Owner

@ivan-hc ivan-hc commented May 4, 2026

fix #2305

ivan-hc added 5 commits May 3, 2026 04:28
Keep dev in sync with main
- _online_check now can alternate between wget and curl
- wget/curl availability is determined directly in _online_check
- move _online_check before dependencies check
- list missing dependencies before installing them, and show the source
- remove appimagetool and other extra dependencies not in "am-utils" if the command is in $PATH
always check "purity" of the AMCACHEPATH directory
Keep dev in sync with main
@ivan-hc ivan-hc changed the title "AM" 9.1.1 "AM" 10.1.1 May 4, 2026
@fiftydinar
Copy link
Copy Markdown
Contributor

fiftydinar commented May 4, 2026

I think it would be good to additionally have notify-send call (if available) when missing crucial dependencies, as it's a very important issue and some users might have am -u run in the background not seeing the terminal message, so it's better to make them aware of it.

@ivan-hc
Copy link
Copy Markdown
Owner Author

ivan-hc commented May 4, 2026

I think it would be good to additionally have notify-send call (if available) when missing crucial dependencies, as it's a very important issue and some users might have am -u run in the background not seeing the terminal message, so it's better to make them aware of it.

done

@fiftydinar
Copy link
Copy Markdown
Contributor

I think it would be good to additionally have notify-send call (if available) when missing crucial dependencies, as it's a very important issue and some users might have am -u run in the background not seeing the terminal message, so it's better to make them aware of it.

done

Hmm, maybe I didn't explain good.

I meant to have `notify-send $" 💀 ERROR! MISSING ESSENTIAL COMMANDS\033[0m: %s\n\n Install the above and try again! \n" "$(echo "curl, wget")"

notify-send "Missing commands will be temporarily downloaded as static binaries:\n\n"
notify-send printf " - %b, downloaded from https://github.com/ivan-hc/am-utils\n" "$name"

and similar

@ivan-hc
Copy link
Copy Markdown
Owner Author

ivan-hc commented May 4, 2026

Hmm, maybe I didn't explain good.

I meant to have `notify-send " 💀 E R R O R ! M I S S I N G E S S E N T I A L C O M M A N D S \0 33 [ 0 m : (echo "curl, wget")"

notify-send "Missing commands will be temporarily downloaded as static binaries:\n\n" notify-send printf " - %b, downloaded from https://github.com/ivan-hc/am-utils\n" "$name"

and similar

It's a vicious circle, then... if a user doesn't have wget or curl, how can they have notify-send? In my opinion, they couldn't even have a Linux distro, let alone AM installed.

@ivan-hc
Copy link
Copy Markdown
Owner Author

ivan-hc commented May 4, 2026

I mean, installing the same app on two levels is unlikely... but something like this is literally absurd.

@fiftydinar
Copy link
Copy Markdown
Contributor

I mean, installing the same app on two levels is unlikely... but something like this is literally absurd.

Ok, it can be ignored

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve AM install script error handling

2 participants