Skip to content

Conversation

@MadsRC
Copy link

@MadsRC MadsRC commented Jul 17, 2015

A makefile for fast & easy install (and removal) of usbkill, it's settingsfile and 2 man pages explaining usbkill and it's settingsfile.

I also created a AUTHORS file with a list of everyone that was listed as cotnributors.

Last but not least, I trimmed the usbkill.py code so there's noi longer any unneeded trailing spaces.

@hephaest0s
Copy link
Owner

Thanks! But, I have two issues that prevent me from accepting this commit into master:

  • For an installer, I want to have `setup.py' install it as a script, as this is the conventional way for installing python programs. Also, I want to keep settings.ini in /etc/
  • You list authors in the authors file, but it is not clear to me that they want to be listed there (I wouldn't). I will ask them first if that is what they want, or they can notify me. You can list you and me there for now, and I will follow up on it.

@ghost
Copy link

ghost commented Jul 29, 2015

Hello,

There is no problem to list me into the AUTHOR file but I think it's a bit useless as the commits are already speaking by themselves.

Regards,
S

Sent from my iPhone

On 29 Jul 2015, at 11:58, Hephaestos [email protected] wrote:

Thanks! But, I have these issues that prevent me from accepting this commit into master:

For an installer, I want to have `setup.py' install it as a script, as this is the conventional way for installing python programs. Also, I want to keep settings.ini in /etc/

You list authors in the authors file, but it is not clear to me that they want to be listed there (I wouldn't). I will ask them first if that is what they want, or they can notify me. You can list you and me there for now, and I will follow up on it.


Reply to this email directly or view it on GitHub.

@MadsRC
Copy link
Author

MadsRC commented Jul 29, 2015

Well, we're going to have to agree to disagree about the Makefile, but it's you're project. If you don't want it, I'll remove it. About the settings file, I'd still recommend having it in /usr/local/etc and then symlinking it in /etc. That way you get to reference /etc/usbkill/settings.ini in your code and be POSIX compliant. POSIX specifies that /etc is for system wide configuration and /usr/local/(etc...) is for local data specific to the host. It would also make the project more compliant towards the hier(7) page.

About the second, for me it's clear that people would want to be listed as contributors, as they already are listed, as @pwnsdx mentioned, in the commits and under "contributors on GitHub. I'll remove people, and have them enter it themselves.

I'll make a new commit later tonight.

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

Successfully merging this pull request may close these issues.

2 participants