<p style="white-space: pre-wrap; word-wrap: break-word;">In general I think spreading out the removals over numerous patches adds little more than effort. Does it matter how we move the files? The end result will be that they are gone from here, and it's not like looking at file removals is confusing, i.e. splitting up doesn't really make anything more easy to read, does it?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Ok, to revert a single repos would be easier with single patches, but I doubt we'll do that, and I doubt even if that it's worth the extra effort (including time spent on moving doc patches that are being written while this refactoring is in limbo)</p><p style="white-space: pre-wrap; word-wrap: break-word;">I accept opinions, but would prefer if we could get this done sooner rather than later, and avoid treading out more bikeshed issues more than we have already...</p><p>Patch set 7:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/11738">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11738/7//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11738/7//COMMIT_MSG@7">Patch Set #7, Line 7:</a> <code style="font-family:monospace,monospace">move OsmoMSC files to osmo-msc.git</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">You are not moving them here, you are removing them from osmo-gsm-manuals. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">(IMHO "move" is better, because that's semantically what we're doing. but either way is fine.)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11738/7//COMMIT_MSG@9">Patch Set #7, Line 9:</a> <code style="font-family:monospace,monospace">Add to osmo-msc.git with I2f9c27f81940b02414f0e618483bffade72e8cf7.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Is this Change-Id part of osmo-msc.git? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">(IMHO also this is fine. but so is pau's version)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11738/7//COMMIT_MSG@14">Patch Set #7, Line 14:</a> <code style="font-family:monospace,monospace">Related: OS#3385</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Depends: osmo-msc. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">if this build breaks without that osmo-msc commit, 'Depends:' is accurate. I guess it's more like 'Related:'?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11738/7/Makefile.am">File Makefile.am:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11738/7/Makefile.am@8">Patch Set #7, Line 8:</a> <code style="font-family:monospace,monospace">SUBDIRS = tests OsmoBSC OsmoBTS OsmoGGSN OsmoGSMTester OsmoHLR OsmoMGCP OsmoMGW OsmoNAT OsmoNITB OsmoPCU \</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Would make sense to split this line into a multiline with \, since we'll be changing it a lot in fol […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">(IMHO this is not needed)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11738">change 11738</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/11738"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-manuals </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib234fe4f4ac36c27a4ad8a8d0050c1d6874232bc </div>
<div style="display:none"> Gerrit-Change-Number: 11738 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 21 Nov 2018 13:03:31 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>