-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Write from Stylus Labs #24555
Add Write from Stylus Labs #24555
Changes from all commits
ea84bca
f3c71f6
7473511
9f49c00
826836a
cfad335
38ff902
5c9dc98
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
# Template file for 'write-styluslabs' | ||
pkgname=write-styluslabs | ||
version=3.0.0 | ||
revision=1 | ||
archs="x86_64" | ||
wrksrc="Write" | ||
depends="SDL2" | ||
short_desc="Word processor for handwriting" | ||
maintainer="Cat Marin <me@jmarin.xyz>" | ||
license="custom:Proprietary" | ||
homepage="http://www.styluslabs.com/" | ||
distfiles="http://www.styluslabs.com/write/write-latest.tar.gz" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As far as I can tell that's how they release packages. I actually tried doing it with a variable and xbps-src doesn't accept There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. They provides versioned tarball, they just don't show you, here is the link for 3.0.0 http://www.styluslabs.com/write/write300.tar.gz There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Yep! Saw that in their response to me email. However they still don't have a license available, which as I understand is necessary in order to proceed. In any case, judging by how many changes I needed to make (and quality concerns of my template) I'm in no way ready to take on maintaining a package. I'll look more into the documentation and other package templates before I do anything else. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nah, packages in void aren't that hard. TBH, it's really just that you chose a slightly problematic package to start on :) |
||
checksum="ec7c8a59f453574aa272d859e4c926eab48122d4965644871bf7cbe3d4703c40" | ||
repository=nonfree | ||
restricted=yes | ||
|
||
do_install() { | ||
local package_location="usr/lib/$pkgname" item | ||
vmkdir usr/share/pixmaps | ||
vcopy Write144x144.png /usr/share/pixmaps | ||
vmkdir usr/share/applications | ||
sed -i "s|/opt/Write/Write|/usr/lib/$pkgname/Write|g" Write.desktop | ||
vcopy Write.desktop /usr/share/applications | ||
vmkdir ${package_location} | ||
chmod +x Write | ||
for item in Write DroidSansFallback.ttf Intro.svg Roboto-Regular.ttf; do | ||
vcopy "Write" "${package_location}" | ||
vcopy "${item}" "${package_location}" | ||
done | ||
vmkdir usr/bin | ||
ln -sfr $DESTDIR/${package_location}/Write $DESTDIR/usr/bin/write-styluslabs | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As the
xlint
CI test shows, a proprietary license declaration requires use ofvlicense
indo_install
orpost_install
to copy the license text into the package. As it stands, I see no license terms attached to this software. You should identify the license terms and be able to fetch the license text from somewhere so we know what terms govern the software.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm currently in the process of contacting Stylus Labs on how they'd like me to proceed here, I'll update my PR when I know.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If they don't make versioned releases. it's unlikely to be accepted as a package here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure how well it fits under quality requirements either.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've sent them an email regarding the issues on their end (including the license), I'll be closing my pull request until they respond/provide versioned downloads and a license.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
http://www.styluslabs.com/write/write300.tar.gz