Skip to content

change to rsync#518

Merged
jennyhickson merged 5 commits intoMetOffice:mainfrom
james-bruten-mo:change_to_rsync
Dec 4, 2025
Merged

change to rsync#518
jennyhickson merged 5 commits intoMetOffice:mainfrom
james-bruten-mo:change_to_rsync

Conversation

@james-bruten-mo
Copy link
Contributor

The patch method of migrating branches has been hitting issues around new files in new directories.
I can't work out a way to do this, so instead we're suggesting that we use rsync instead. If branches are created from an equivalent point, this should result in the same outcome as previously.

Copy link
Collaborator

@jennyhickson jennyhickson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working out a new process. A couple of questions...

commit on git - tags will have the same name on both, revisions of specific
commits will need to be manually aligned with a commit hash by comparing
commit messages.
The process below involves rsyncing the changes in your fcm working copy to
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think its still good to have some words about the git_migration tag and others - maybe in a tip underneath this important box?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I read the entirety of the tip box correctly. I've added that stuff back in

#. Create a patch file from your new branch at the migration point. To do this
you will need to know the branch point of your fcm branch, which can be found
from the ``fcm binfo`` command
#. Get an export of the branch you wish to migrate (the one above in this case).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious as to why an export not a checkout? Particularly if you then delete the .svn directory doesn't that make them basically the same?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thinking about it we can probably just exclude the .svn dir below as well. Removed this step and added that there

@jennyhickson jennyhickson added this to the Git Migration milestone Dec 4, 2025
Copy link
Contributor Author

@james-bruten-mo james-bruten-mo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Jenny, triggered a few changes

commit on git - tags will have the same name on both, revisions of specific
commits will need to be manually aligned with a commit hash by comparing
commit messages.
The process below involves rsyncing the changes in your fcm working copy to
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I read the entirety of the tip box correctly. I've added that stuff back in

#. Create a patch file from your new branch at the migration point. To do this
you will need to know the branch point of your fcm branch, which can be found
from the ``fcm binfo`` command
#. Get an export of the branch you wish to migrate (the one above in this case).
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thinking about it we can probably just exclude the .svn dir below as well. Removed this step and added that there

Copy link
Collaborator

@jennyhickson jennyhickson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. One last thing...


.. important::

For convenience, the ``git_migration`` tag is provided as a branch point in
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please can you swap the paragraph order on these two? What is currently the second paragraph gives the more important detail - the rest is supporting information.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem :)

@jennyhickson jennyhickson merged commit e76bdec into MetOffice:main Dec 4, 2025
2 checks passed
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