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

Packaging #26

Open
Thaodan opened this issue Jul 10, 2022 · 9 comments
Open

Packaging #26

Thaodan opened this issue Jul 10, 2022 · 9 comments

Comments

@Thaodan
Copy link
Contributor

Thaodan commented Jul 10, 2022

Hey,

I have a few questions regarding packaging this app.

  • Why does mix try to install hex and rebar even thou it is already already installed with elixir?
  • Can I make mix deps.get stop asking questions?
@progval
Copy link
Owner

progval commented Jul 10, 2022

the --overwrite flag of mix release skips the question, at least on rebuilds. I don't know about the other question

@Thaodan
Copy link
Contributor Author

Thaodan commented Jul 10, 2022 via email

@progval
Copy link
Owner

progval commented Jul 10, 2022

Do you have any documentation of what this does and how to use it?

@Thaodan
Copy link
Contributor Author

Thaodan commented Jul 10, 2022 via email

@Thaodan
Copy link
Contributor Author

Thaodan commented Jul 10, 2022

Are you also interested into have a systemd unit saved in this repo?

@progval
Copy link
Owner

progval commented Jul 10, 2022

Ugh that looks annoying. There's already a systemd unit in https://github.com/progval/matrix2051/blob/main/INSTALL.md ; feel free to improve it or suggest alternatives

@Thaodan
Copy link
Contributor Author

Thaodan commented Jul 10, 2022

@progval
Copy link
Owner

progval commented Jul 10, 2022

Oh I didn't realize you were actually packaging it in AUR.

It looks good to me.

There are many redundant options in this service. eg. ProtectSystem=strict and ProtectHome=true are implied by DynamicUser=true.
Is it standard practice to include them anyway?

progval added a commit that referenced this issue Jul 10, 2022
@Thaodan
Copy link
Contributor Author

Thaodan commented Jul 11, 2022

Oh I didn't realize you were actually packaging it in AUR.

It looks good to me.

There are many redundant options in this service. eg. ProtectSystem=strict and ProtectHome=true are implied by DynamicUser=true. Is it standard practice to include them anyway?

Oh thanks, it is not practice to include redundant options however if for some reasons a non dynamicuser setting would be ued they still to make sense to include.

Thaodan added a commit to Thaodan/matrix2051 that referenced this issue Jul 11, 2022
progval pushed a commit that referenced this issue Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants