This page is about migrating FPC from SVN to git
- 1 Concerns/Questions
- 2 Tool for migration
- 3 Work to do
What part of SVN to migrate ?
- More is better.
Jonas has a very complete git mirror of the SVN+CVS part.
(care needs to be taken: there used to be a time when copyrighted code was checked in)
A first test conversion by Florian using subgit was attempted: completed in 5 hours 1, creash. Looks OK.
- In order to save on diskspace, find ways to tell user how to clone only a part.
git clone --depth 1 to get only the latest revision (with fetch, later on more revisions can be fetched if needed)
The fpc build repository uses svn:external references.
Git has modules, which is in essence the same. This needs to be properly set up.
Branching model ?
Various workflow models exist:
Also known as "git-flow" model.
- A simple and logical workflow that plays to the strengths of Git - branching and merging.
- 'master', 'develop' and 'release' are good branch names which immediately makes it obvious what they are for. Many developers clone a repo and want or expect a stable version. This workflow allows just that - the 'master' branch is the default branch, and is always the latest stable release of the product.
- Features or multi-commit (complex) bug fixes show a clear commit history or related commits, that is easily tracked, viewed or even rolled-back in the commit history using a tool like gitk.
- It works only well if the the whole release policy of FPC is changed. This is something very likely not going to happen.
- cluttered and basically unreadable history
- doubled regression testing effort during development of new stuff
- test branch before merge
- test branch after merge, as merging could have broken things
micro management of branches: every single bug fixes requires and results in a new branchThis is simply not the case. single bug fixes can be committed into the 'develop' branch as a linear history. Only more complex (multi-commit) fixes or features will result in a parallel history or development, but quickly merged back into 'develop'. --Graeme (talk) 18:50, 17 December 2017 (CET)
You do not see the problem: if it is in develop, it will never make it into master without merging whole develop into master (i.e. in terms of FPC releases this would mean a new minor release x.y+1.0). So each bug fix has to go into an own hotfixes branch (e.g. hotfix_bug12345) which is branched from master and merged first into develop. If it works, it will be merged back into master. --FPK (talk) 22:58, 17 December 2017 (CET)
- One has to know before pushing how invasive a change is because it has to go in the right branch. For compiler development this is very cumbersome.
Also known as the anti-"A successful Git branching Model".
- clear and straight history
- no time consuming micro management of branches
- history and logs not cluttered with merge commits
- no breakage by wrong conflict resolving during merge commits by less experienced users
- Git was designed to work well with branches and handle merging as a common task. This workflow suggests merging as a bad idea, which is crazy.
- This model recommends that a human must now keep track of which commits to cherry-pick into other branches. This is bounds to fail in the long run and vital commits will be missed at some point. Merging two or more branches on the contrary will automatically handle all commits in a branch seamlessly, so why not use what Git was designed to do well.
- This model suggests that you can also cherry-pick from an unstable branch back into a stable release branch. This is just the wrong way round.
- This model sees some of its own flaws and suggests a 3rd party tool to help keep track of things like cherry-picking. This simply isn't needed if you use a better workflow.
- It suggests that 'rebase' get used often. This has always been frowned upon in the Git community, and it is widely known to use rebase sparingly as it rewrites history and doesn't result in a logical evolution of the code history.
- This worklfow suggests that a merge commit is a bad thing. A merge commit is a commit like any other. It can be a simple commit (with no conflicts), or can be a commit that resolves conflicts (thus code changes were required - which this commit records). Nothing bad about that.
- The git project itself gets an extreme amount of contributions via pull requests and emailed patches. Their workflow model clearly works well and uses branching and merging extensively - even though the commit history looks quite scary. :-)
- // to be written
User management ?
Git has no concept of users. To manage permissions on a server, a separate program is needed.
- uses git repo for administration
- No server binary
- No web interface
- administration needs ssh key, only ssh possible.
- Web integration ?
- Web based
- Fine tuning possible
- Separate config
- Requires running binary all the time, on a separate port.
What about Lazarus ?
Tool for migration
- very flexible
- very fast (remote cloning of the whole FPC repository takes 5-6 hours)
- external java-based tool, so some dependencies
- included in git it self
- less flexible
Work to do
Migrate SVN repo.
- 2017-12-16: A first test conversion by Florian using subgit was attempted: completed in 5 hours, 1 crash. Looks OK
Branch mapping for subgit
Proposal for branch mapping
excludeBranches = branches/aspect excludeBranches = branches/avr32 excludeBranches = branches/blaise excludeBranches = branches/cpstr excludeBranches = branches/cpstrnew excludeBranches = branches/ctypes excludeBranches = branches/dodi excludeBranches = branches/florian excludeBranches = branches/foxsen excludeBranches = branches/fcl-web_joost excludeBranches = branches/generics excludeBranches = branches/genfunc excludeBranches = branches/janbruns excludeBranches = branches/linker excludeBranches = branches/newthreading excludeBranches = branches/peterjan excludeBranches = branches/ssa excludeBranches = branches/svenbarth/classhelpers excludeBranches = branches/svenbarth/collector excludeBranches = branches/svenbarth/generics excludeBranches = branches/svenbarth/misc excludeBranches = branches/svenbarth/nativent excludeBranches = branches/svenbarth/packages excludeBranches = branches/tg74/rtl excludeBranches = branches/tg74/tests excludeBranches = branches/tg74/utils excludeBranches = branches/unitrw excludeBranches = branches/wkrenn excludeBranches = branches/FIXES_2_2 branches = branches/merged/*:refs/heads/merged/* branches = branches/joost/*:refs/heads/joost/* branches = branches/laksen/*:refs/heads/laksen/* branches = branches/maciej/*:refs/heads/maciej/* branches = branches/olivier/*:refs/heads/olivier/* branches = branches/paul/*:refs/heads/paul/* branches = branches/*:refs/heads/svn/* branches = branches/svenbarth/*:refs/heads/svenbarth/* branches = branches/tg74/*:refs/heads/tg74/*