Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Debian postinst/preinst #35

Closed
ahall opened this issue Sep 15, 2013 · 6 comments
Closed

Debian postinst/preinst #35

ahall opened this issue Sep 15, 2013 · 6 comments

Comments

@ahall
Copy link

ahall commented Sep 15, 2013

I've started looking at sbt-native-packager and couldn't figure out how to add a postinst script to my packages. Is it possible to do at the moment?

Thanks in advance!
@jsuereth
Copy link
Member

Apparently not, currently. YOu might be able to sneak them in via the debianMaintainerScripts setting, but that's undocumented.

Pull request welcome, if you have time. Otherwise I'll circle back here when I get a chance.

@muuki88
Copy link
Contributor

muuki88 commented Sep 21, 2013

There is an example here

@ahall
Copy link
Author

ahall commented Sep 21, 2013

Thanks guys!

jsuereth added a commit that referenced this issue Oct 25, 2013
@jsuereth
Merge pull request #53 from muuki88/master
8f94469 
First commit on #35
@muuki88 muuki88 mentioned this issue Oct 30, 2013
Merged
jsuereth added a commit that referenced this issue Oct 31, 2013
@jsuereth
Merge pull request #57 from muuki88/master
dc93fa4 
Debian scripts are now picked up from default folder #35
@muuki88
Copy link
Contributor

muuki88 commented Oct 31, 2013

I think we can close this for now. Documentation has to be added (however I'm not quite familiary with Sphinx).

Replacements options are

author - author of this project
descr - description
name - normalized name of the app
version - app version

Default folder is src/debian/DEBIAN. An example prerm could look like

echo "removing logs for ${{name}} with version ${{version}}"
rm -rfv /var/log/${{name}}

@jsuereth
Copy link
Member

cool. Let's leave it open and I'll handle the docs before we close :)

@muuki88
Copy link
Contributor

muuki88 commented Oct 31, 2013

deal 👍

This was referenced Oct 31, 2013
Merged
Closed
jsuereth added a commit that referenced this issue Nov 1, 2013
@jsuereth
Add documentation for maintenance script additions.
2ab67c5 
Note: Documentation probably needs work overall.  Probably a getting started guide and
more detailed information.

Docs for #35 fix.
@jsuereth jsuereth closed this as completed Nov 1, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jsuereth @ahall @muuki88