Adding reMarkdown to vellum#302
Conversation
|
/publish-testing |
| maintainer="esclee <[email protected]>" | ||
| pkgname=remarkdown | ||
| pkgver=0.1.0 | ||
| pkgrel=1 |
There was a problem hiding this comment.
| pkgrel=1 | |
| pkgrel=0 |
|
Packages published to testing repo:
Install with: |
| unpack() { | ||
| case "$CARCH" in | ||
| aarch64) unzip -o "$srcdir"/reMarkdown-arm64-$pkgver.zip -d "$srcdir"/rmd ;; | ||
| armv7) unzip -o "$srcdir"/reMarkdown-armv7h-$pkgver.zip -d "$srcdir"/rmd ;; | ||
| esac | ||
| } |
There was a problem hiding this comment.
Since this just appears to be unifying the directory it unpacks to, is it really worth it to override extract instead of just doing a cd to the correct location in package?
| postinstall() { | ||
| echo "" | ||
| echo "==================================================" | ||
| echo " NOTE: To use reMarkdown, you must refresh AppLoad" | ||
| echo "==================================================" | ||
| echo "" | ||
| } |
There was a problem hiding this comment.
@rmitchellscott we keep on having packages all duplicate this message all over the place, which results in multiple instances of this message if you install more than one application. Is is something we really care to present to users this way via vellum? Especially now that there is more than one launcher?
There was a problem hiding this comment.
I'm fine with removing these messages, as we don't notify that xovi needs to be restarted for every QMD, etc.
There was a problem hiding this comment.
Sounds good, I can open a PR to clean up all the other packages. Ideally we work on using triggers in packages to unify messages like this, or better yet, trigger the reload with them instead.
Adding reMarkdown to vellum