-
Notifications
You must be signed in to change notification settings - Fork 25
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 deb package generation #41
base: master
Are you sure you want to change the base?
Conversation
DEBIAN/control
Outdated
Architecture: amd64 | ||
Maintainer: jrandiny <[email protected]> | ||
Depends: ibus,python3,gir1.2-ibus-1.0,python3-levenshtein | ||
Description: Ibus input method for entering emoji by name |
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.
Ibus
should be IBus
.
README.md
Outdated
@@ -28,6 +28,8 @@ Installing | |||
|
|||
To install, type `make install`. If your ibus isn't on /usr/share/ibus, or you want to install to /usr/local, you can pass any of `PREFIX`, `DATADIR`, and `SYSCONFDIR` to `make`. You can also pass `DESTDIR` to aid in packaging, or `PYTHON` to use a different Python executable. | |||
|
|||
For Ubuntu you can create a `.deb` file by running `make deb` and install with `sudo apt install ./<created_deb_filename>.deb` |
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.
Maybe mention Debian before Ubuntu since I assume this will work on all Debian-derived distributions.
deb: | ||
make install DESTDIR=./deb-output/ | ||
cp -r DEBIAN deb-output | ||
sed -i "s/<currentversion>/$(CURRENTDATE)/" deb-output/DEBIAN/control |
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.
Maybe the makefile should include a proper version, and if you're building from git, the sha1 can be used, appended to the version.
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.
The problem with using sha1 for the version is that dpkg cannot determined which package is newer. How about date of last commit followed by the sha1?
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.
why not a proper version number for the deb package? i don't think a new deb should be generated per commit anyway
This PR will add the ability to generate
.deb
file for better package management