On Tuesday, 9 September 2014 at 09:59:37 UTC, Dragos Carp wrote:

ne important (in my opinion) disadvantage misssing: considerably harder PR maintenance (missing automatic categorization and separation between phobos / dmd teams). Being forced to look ar DMD pull requests to do Phobos maintenance will add lot of context overhead for me.

This kind of attitude worries me: every team works happily in its own cozy repository and it doesn't feel responsible about the other repos. The consequence is that, while breeding a new release, the release manager MUST take most of the burden about coordinating the teams and their work. This is a very complex task which requires insight knowledge in all 3 repos.

This is not about happiness or zone of comfort - I am simply not competent to judge even most trivial DMD pull request as opposed to Phobos ones. Team separation exists for a reason - skills and knowledge required are totally different. Some developers do both but I doubt it is even the majority of active maintainers. By merging repos you are likely to discourage me from paying attention at all, not to invest huge amount of effort in increasing DMD proficiency.

I am also not sure what what coordination you are speaking about. Integrity of 3 repos is verified by auto-teser. Cross-repo issues are rare and usually handled by those who has merge access to both. Release manager shouldn't normally need any detailed knowledge of repo internal development and just tag certain branch heads. If this is not the case we have some hidden problem I am not aware of.

Having 3 repos, one team is not encouraged to help the other team to get the release out the door. This is because one team have a higher bar to help the other: it needs to checkout the right version of the foreign repo, manage own branches in the foreign repo, etc. Maybe this is the reason of multiple regression in the last release or the "virtual keyword surprise" (at least for some people).

It feels like you have imagined some process that does not really match reality :( Explicit tracking of matching version is never needed - it is either branch head for all repos, or identically named tags.

I have already explained above some of the real reasons that make closer collaboration close to impossible.

I think that a 2 phase process similar to the Linux kernel release process [1] would work much better and motivates all the teams to help with the release:

Phase 1. Merge window: merge PRs containing new features
Phase 2. Test and stabilization: merge bugfix PRs only

Ideally the first phase is quite short (1 week for example) and it can be close supervised (or even executed??) by Walter and/or Andrei.

The second phase will continue until the acceptance criteria are met, the teams having a focus on the release (they get no new goodies till the test phase is done).

The normal development happens somehow offline by preparing quality PRs to be merged in phase 1.


This all sounds like solving the problem that doesn't exist instead of addressing the real ones (lack of clear documented rules for release packaging and awkward cherry-pick based release branch maintenance)

Reply via email to