atar-axis / xpadneo

Advanced Linux Driver for Xbox One Wireless Controller (shipped with Xbox One S)
https://atar-axis.github.io/xpadneo/
GNU General Public License v3.0
1.87k stars 110 forks source link

Consider cases for `--depth=1` when cloning #420

Closed kakra closed 4 months ago

kakra commented 1 year ago

I feel like this would cause more harm than benefit. People are encouraged to install via a distribution package. Changing the clone depth would prevent people from properly using branches when asked for in the issues. So I'd argue that people who got that far down are more likely to do development or issue reporting stuff. A size delta of 20 MB is just too small to outperform the possible drawbacks. Yes, it's like 85% less storage but in absolute numbers it's just 20 MB - that's a nothingness in today's systems.

If anything like this should go into the documentation, it should be for the stable branch. Unfortunately, this project doesn't have a linear stable branch, we branch off the current development branch when a development feature is ready and backport patches much like the kernel does. This would mean to change the docs on each stable cycle to the new stable branch name.

I'll leave this open as a reminder to amend the docs to encourage people to use their distribution package or a stable release.

Originally posted by @kakra in https://github.com/atar-axis/xpadneo/issues/412#issuecomment-1474807492

kakra commented 1 year ago

@Apacelus I've opened a new issue to follow up on your idea at a later time because at some time in the future it might bring benefits.

kakra commented 4 months ago

Thinking about this, I came to this conclusion:

Thus, I'm closing this.