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.

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

ok? #21

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@oleg-pahl
Copy link
Contributor

oleg-pahl commented Feb 15, 2019

No description provided.

oleg-pahl added some commits Feb 15, 2019

@oleg-pahl

This comment has been minimized.

Copy link
Contributor Author

oleg-pahl commented Feb 15, 2019

fix debhelper compatibility

@alexander-naumov
Copy link
Owner

alexander-naumov left a comment

Please always provide some short explanation...

In this case: 6b906fa
Why do you think it's better to do it in prerm and not in postrm?

@alexander-naumov

This comment has been minimized.

Copy link
Owner

alexander-naumov commented Feb 15, 2019

Yes, this commit 551d4d7 is ok.

@oleg-pahl

This comment has been minimized.

Copy link
Contributor Author

oleg-pahl commented Feb 18, 2019

Please always provide some short explanation...

In this case: 6b906fa
Why do you think it's better to do it in prerm and not in postrm?

Postinst corresponds to prerm, not postrm

@alexander-naumov

This comment has been minimized.

Copy link
Owner

alexander-naumov commented Feb 18, 2019

Postinst corresponds to prerm, not postrm

Postinst? There is nothing about postinst. We're talking about prerm VS postrm...

Please read official manual:
https://www.debian.org/doc/manuals/debian-faq/ch-pkg_basics.en.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.