-
Notifications
You must be signed in to change notification settings - Fork 15
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
Repo branch mess #19
Comments
Well, it's not exactly random. bug fixes to master and new features to develop for further evaluation and later merging into master... |
Well, I suppose, but then you have to deal with the awkward situation of |
Not to worry, I'll include those new changes now they are in master and submit a new PR to bring develop up to date. |
The issue is more one of making us able to test a separate branch while the master branch is supposed to be a known good working version. But hotfixes in master, features in develop is always a good practice. People in general expect a master branch to be somewhat usable and the develop branch an experimental one. |
The way I usually handle experimental stuff is just on a per-PR basis. Test the PR on its own, and only merge it once we know it's good. Do as you wish, I'm just sharing the workflow that works well for me, since things are looking a little messy here. |
I try to always develop also up to date wiht master. Hotfixes should not go into develop as it might bring with them experimental not yet completed stuff if both fixes and features live in develop - if you do it that way you absolutely need to also do all new features in separate branches as well. |
The reason for there being only the g13 package on AUR is that I was not the one creating the package in the first place. I have just nursed it along following a path of little resistance - while refactoring the g13 daemon as it was depending on boost which I loathe.
I am happy that you guys got involved but now that you are - the develop branch and master branch can go out of sync. For now - lets do crazy stuff in either develop or feature specific branches and keep them in sync with master at least on release tags so they never fall far behind. Does this sound ok? |
Basing the current AUR package on release tags doesn't fit with the Also, thanks for getting rid of boost, I despise that library as well. Why does anyone use it? libstd provides basically everything you need! My experience with boost has been nothing but long compile times, excessive gcc memory usage, and compatibility problems with different versions of boost. |
Yeah the package has been left as is but that is mostly because I have not liked the idea of maintaining more than one package for it - coding is a lot more fun than mantaining ;-) I could not agree more on the subject of boost. It had a use before libstd got up to like C++17, maybe even C++11 already. Boost has given me so much grief over the years so I try to kill it on sight. |
Okay, we can't keep merging things randomly into
master
ordevelop
, this is just a mess now. The way this model is supposed to work is all new features go todevelop
first, thendevelop
is merged intomaster
when it's time for a release. Alternatively, you could simply merge everything intomaster
and use tags to specify "stable" releases. This model is probably easier to stick to, and seems to be the way many projects go.Additionally, you could have two AUR packages: one which builds from a specified version tag, and another which just builds the latest master. The convention on the AUR is to name these
g13
andg13-git
respectively. Both are still compiled from the Git repo, but the latter implies it's the very latest code, while the former is a specific release version.The text was updated successfully, but these errors were encountered: