Skip to content

Updated error msg printf for "am -u am".#2092

Merged
ivan-hc merged 1 commit intoivan-hc:devfrom
vishnu350:main
Feb 18, 2026
Merged

Updated error msg printf for "am -u am".#2092
ivan-hc merged 1 commit intoivan-hc:devfrom
vishnu350:main

Conversation

@vishnu350
Copy link
Contributor

Just a simple fix I made a while back to improve the error message explained in issue #2083.

Thanks.

@ivan-hc ivan-hc changed the base branch from main to dev February 18, 2026 16:15
@ivan-hc
Copy link
Owner

ivan-hc commented Feb 18, 2026

Good, I'll merge in dev for review, but don't worry. It is a welcomed change.

@ivan-hc ivan-hc merged commit ac7aeec into ivan-hc:dev Feb 18, 2026
@ivan-hc
Copy link
Owner

ivan-hc commented Feb 18, 2026

I have changed it like this

[ "$arg" = "$AMCLI" ] && arg_autoupdatable=", use \"-s\" instead"
  • quotes on "$arg" to prevent expanding
  • $AMCLI instead of am, to made it work also with AppMan
  • "-s", without specify the CLI

@ivan-hc
Copy link
Owner

ivan-hc commented Feb 18, 2026

@vishnu350 welcome on board!

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.

2 participants