run-session: fix evaluation of $cmd variable

for some reason, it has worked as is on multiple instances that i've tried, but recently doesn't work. either way, the extra reevaluation of $cmd was not correct

approved this merge request

forgot to mention here ... i would suggest this is included in v24.12 as well (meaning the packaging in pmaports would need updating as well)

In what circumstances does the current behaviour cause problems? What does this fix?

added status::mr-stale label

Sorry to bother you @sicelo,

but we've detected that this merge request hasn't seen any recent activity. If you need help, want to discuss your approach with developers, or you are all done and this is waiting for review, you can ping @postmarketOS. You can also ask on matrix in #devel:postmarketos.org or #postmarketos-devel on OFTC.

Thanks for your contribution.

approved this merge request

Looking at it again, the commit message explains what this does.

* Drop extraneous $cmd variable reevaluation
* Update the comment for this feature. The session profile is evaluated
  only later in the script, so the environment variable should be set
  in the regular profile

LGTM, thanks!

added 2 commits

• 3c0a1e18 - 1 commit from branch postmarketOS:master
• 7fe439fd - run-session: fix evaluation of $cmd variable (MR 16)

Compare with previous version

merged