[gentoo-dev] [PATCH 00/16] devmanual: update the docs for post git-migration #558642

[gentoo-dev] [PATCH 00/16] devmanual: update the docs for post git-migration #558642

[Gokturk Yuksek]

Hi,

The following patch series attempt to update devmanual to replace the instances
of CVS with their git equivalents. I tried to keep the changes small in favor
of updating the docs over multiple iterations of patch series. The intention is
to get the bare bone changes in so that the quiz questions can be answered.

Currently not included in the series are:
- References to CVS in sections about herds since they will have to be
  rewritten per GLEP 67 soon anyway.
  See: https://bugs.gentoo.org/show_bug.cgi?id=572144
- I am not sure about what to do with the subsection
  Ebuild Functions/Miscellaneous Files/ChangeLog. The ChangeLog files
  still exist in the rsync tree, however they are of little importance
  in terms of development.

Gokturk Yuksek (16):
  general-concepts/manifest: drop the use of "CVS keyword expansion"
    #558642
  general-concepts/mirrors: substitute "CVS" with "the git tree" #558642
  general-concepts/tree: substitute "CVS" with "git" #558642
  general-concepts/tree: replace the mention of ChangeLog #558642
  ebuild-writing/misc-files: replace the code for cvs commit with git
    #558642
  ebuild-writing/user-submitted: do not put user information in
    ChangeLog #558642
  appendices/editor-configuration/emacs: remove the CVS related setting
    #558642
  ebuild-maintenance: rewrite the text on adding binary files to the
    tree #558642
  ebuild-maintenance: rewrite the subsection on commit policy for git
    #558642
  ebuild-maintenance: rewrite the subsection on upgrading ebuilds for
    git #558642
  ebuild-maintenance: rewrite the subsection on moving ebuilds for git
    #558642
  ebuild-maintenance: rewrite the subsection on removing ebuilds for git
    #558642
  ebuild-maintenance: rewrite the subsection on removing packages for
    git #558642
  ebuild-maintenance: replace "cvs commit" with "git commit" #558642
  general-concepts/git-to-rsync/diagram: update the description for git
    #558642
  general-concepts/git-to-rsync/diagram: update the description for git
    #558642

 appendices/editor-configuration/emacs/text.xml |  2 +-
 ebuild-maintenance/text.xml                    | 77 ++++++++++----------------
 ebuild-writing/common-mistakes/text.xml        |  2 +-
 ebuild-writing/misc-files/metadata/text.xml    |  3 +-
 ebuild-writing/user-submitted/text.xml         |  8 ++-
 general-concepts/git-to-rsync/diagram.svg      |  2 +-
 general-concepts/manifest/text.xml             |  4 +-
 general-concepts/mirrors/diagram.svg           |  2 +-
 general-concepts/mirrors/text.xml              |  2 +-
 general-concepts/tree/text.xml                 |  4 +-
 10 files changed, 44 insertions(+), 62 deletions(-)

-- 
2.4.10

[gentoo-dev] [PATCH 01/16] general-concepts/manifest: drop the use of "CVS keyword expansion" #558642

[Gokturk Yuksek]

With the git migration no CVS keyword expansions are happening. However,
Manifest file may still need to be regenerated.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/general-concepts/manifest/text.xml b/general-concepts/manifest/text.xml
index 27ebdef..12961da 100644
--- a/general-concepts/manifest/text.xml
+++ b/general-concepts/manifest/text.xml
@@ -18,8 +18,8 @@ The <c>Manifest</c> may also be digitally signed.
 
 <p>
 To generate the <c>Manifest</c>, use <c>ebuild foo.ebuild manifest</c>. When
-committing, the <c>Manifest</c> file must be regenerated to handle any CVS keyword
-expansion changes <d/> <c>repoman</c> will do this automatically.
+committing, the <c>Manifest</c> file must be regenerated to handle any
+changes <d/> <c>repoman</c> will do this automatically.
 </p>
 </body>
 </section>
-- 
2.4.10

[gentoo-dev] [PATCH 02/16] general-concepts/mirrors: substitute "CVS" with "the git tree" #558642

[Gokturk Yuksek]

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/general-concepts/mirrors/text.xml b/general-concepts/mirrors/text.xml
index e860988..6e3386a 100644
--- a/general-concepts/mirrors/text.xml
+++ b/general-concepts/mirrors/text.xml
@@ -55,7 +55,7 @@ On rare occasions you may need to replace a file that is already mirrored.  In t
 follows:
 <ol>
   <li>Put a copy of the new distfile on dev.gentoo.org into /space/distfiles-local</li>
-  <li>commit the new manifest to CVS</li>
+  <li>commit the new manifest to the git tree</li>
   <li>wait</li>
 </ol>
 After a few hours a cron job on dev.gentoo.org will fetch the file and replace the version on the
-- 
2.4.10

[gentoo-dev] [PATCH 03/16] general-concepts/tree: substitute "CVS" with "git" #558642

[Gokturk Yuksek]

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/general-concepts/tree/text.xml b/general-concepts/tree/text.xml
index 022a1ea..6f59f09 100644
--- a/general-concepts/tree/text.xml
+++ b/general-concepts/tree/text.xml
@@ -59,11 +59,11 @@ The basic layout of the portage tree is as follows:
   <li>Scripts directory, <c>scripts/</c></li>
   <li>
   Distfiles directory, <c>distfiles/</c>. This is not included in the main
-  CVS tree, but it will be found on most user systems.
+  git tree, but it will be found on most user systems.
   </li>
   <li>
   Packages directory, <c>packages</c>. Again, this is found on user systems but not
-  in the main CVS tree.
+  in the main git tree.
   </li>
 </ul>
 </body>
-- 
2.4.10

[gentoo-dev] [PATCH 04/16] general-concepts/tree: replace the mention of ChangeLog #558642

[Gokturk Yuksek]

Package updates do not require changing of the ChangeLog file. Package
updates by users that do not mention what has changed is still
considered a common mistake however.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-writing/common-mistakes/text.xml b/ebuild-writing/common-mistakes/text.xml
index 1718d55..1716e5d 100644
--- a/ebuild-writing/common-mistakes/text.xml
+++ b/ebuild-writing/common-mistakes/text.xml
@@ -468,7 +468,7 @@ of the application, if any exists.
 </body>
 </subsection>
 <subsection>
-<title>Package updates without changing the ChangeLog</title>
+<title>Package updates without explaining what has changed</title>
 <body>
 
 <p>
-- 
2.4.10

[gentoo-dev] [PATCH 05/16] ebuild-writing/misc-files: replace the code for cvs commit with git #558642

[Gokturk Yuksek]

Replace "cvs commit" with the equivalent "git add && git commit" version.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-writing/misc-files/metadata/text.xml b/ebuild-writing/misc-files/metadata/text.xml
index e506b1c..a8beacc 100644
--- a/ebuild-writing/misc-files/metadata/text.xml
+++ b/ebuild-writing/misc-files/metadata/text.xml
@@ -532,7 +532,8 @@ is currently:
 <pre>
 xmllint --noout --valid metadata.xml
 glep31check metadata.xml
-cvs commit -m "Adding category metadata.xml for my-category" metadata.xml
+git add metadata.xml
+git commit --gpg-sign -m "Adding category metadata.xml for my-category"
 </pre>
 
 </body>
-- 
2.4.10

[gentoo-dev] [PATCH 06/16] ebuild-writing/user-submitted: do not put user information in ChangeLog #558642

[Gokturk Yuksek]

ChangeLog files are auto-generated by infra post git-migration.
For user-submitted ebuilds, include the user information in the commit
message instead. Mention that user submitted git patches and pull
requests preserve the authorship information by default.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-writing/user-submitted/text.xml b/ebuild-writing/user-submitted/text.xml
index ef80c0d..22f8fa8 100644
--- a/ebuild-writing/user-submitted/text.xml
+++ b/ebuild-writing/user-submitted/text.xml
@@ -21,9 +21,11 @@ The user-submitted ebuild must not contain custom headers like this:
 </pre>
 
 <p>
-Such information should be added to the <c>ChangeLog</c> instead.
-The first <c>ChangeLog</c> entry should always credit the user who
-submitted the ebuild, with their full name and e-mail address.
+Such information should be included in the git commit message instead.
+Note that ebuilds received in the form of git patches or pull requests
+will have the user as the commit author by default, in which case
+including the user information in the commit message explicitly
+may not be necessary.
 </p>
 
 <p>
-- 
2.4.10

[gentoo-dev] [PATCH 07/16] appendices/editor-configuration/emacs: remove the CVS related setting #558642

[Gokturk Yuksek]

Setting vc-cvs-stay-local to nil causes VC to query the remote repository.
Git is a distributed VCS. As such no equivalent command exists for git.
Remove the CVS related setting.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/appendices/editor-configuration/emacs/text.xml b/appendices/editor-configuration/emacs/text.xml
index 55fade0..37c5504 100644
--- a/appendices/editor-configuration/emacs/text.xml
+++ b/appendices/editor-configuration/emacs/text.xml
@@ -48,7 +48,7 @@ automatically check for the existence of it and ask you to add one.
 <p>
 Other useful settings can be disabled backup files
 (by <c>(setq make-backup-files nil)</c> and
-<c>(setq vc-cvs-stay-local nil)</c>), so you don't clutter CVS
+so you don't clutter the git repository
 directories and confuse repoman with it (by adding unnecessary entries
 into a Manifest file e.g.). Emacs can even contact the outside world
 by using the X servers clipboard abilities when yanking, which is
-- 
2.4.10

[gentoo-dev] [PATCH 08/16] ebuild-maintenance: rewrite the text on adding binary files to the tree #558642

[Gokturk Yuksek]

Remove the paragraph about adding binary files to CVS.
Replace the instances of CVS with git.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-maintenance/text.xml b/ebuild-maintenance/text.xml
index ac68dcd..10709ea 100644
--- a/ebuild-maintenance/text.xml
+++ b/ebuild-maintenance/text.xml
@@ -42,22 +42,9 @@ above 20KB) which should be distributed as tarballs via the
 <uri link="::general-concepts/mirrors/#suitable-download-hosts">Gentoo mirror system</uri>
 so that people do not waste excessive amounts of bandwidth and hard drive
 space. Also, you should not add binary (non-ASCII) files to the
-Portage CVS tree. If you need to do this in another CVS tree, for
-example, if you need to add a small PNG graphic for whatever reason,
-be sure to add it to CVS by using the <c>-kb</c> option, like so:
-</p>
-
-<pre caption="Adding binary files to CVS">
-# <i>cvs add -kb myphoto.png</i>
-</pre>
-
-<p>
-The <c>-kb</c> option tells CVS that <path>myphoto.png</path> is a binary
-file and should be treated specially.  For example, merging the 
-differences between two different versions of this file should not be 
-allowed to happen, for obvious reasons.  Also, speaking of merging 
+git tree. Also, speaking of merging
 changes, any patches you add to Portage should generally <e>not</e> be 
-compressed.  This will allow CVS to merge changes and correctly inform 
+compressed.  This will allow git to merge changes and correctly inform
 developers of conflicts.
 </p>
 
@@ -149,7 +136,7 @@ simply, <path>1.0-gentoo.diff</path>.  Also note that the
 <path>gentoo</path> extension informs people that this patch was created 
 by us, the Gentoo Linux developers, rather than having been grabbed from a 
 mailing list or somewhere else. Again, you should not compress these 
-patches because CVS does not play well with binary files.
+patches because git does not play well with binary files.
 </p>
 
 <p>
-- 
2.4.10

[gentoo-dev] [PATCH 09/16] ebuild-maintenance: rewrite the subsection on commit policy for git #558642

[Gokturk Yuksek]

Remove the mention of updating the ChangeLog.
Remove the mention of file conflicts during commit as CVS keeps track of
files whereas git keeps track of changesets.

Add a note regarding commit atomicity: Git commits are always atomic, so
combining package.mask/license changes with ebuild changes in the same
commit does not violate atomicity. However, there is no way to accomplish
this using repoman commit.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-maintenance/text.xml b/ebuild-maintenance/text.xml
index 10709ea..2d46749 100644
--- a/ebuild-maintenance/text.xml
+++ b/ebuild-maintenance/text.xml
@@ -96,7 +96,7 @@ work on those architectures.
 </subsection>
 
 <subsection>
-<title>CVS Commit Policy</title>
+<title>Git Commit Policy</title>
 <body>
 
 <ul>
@@ -105,15 +105,18 @@ work on those architectures.
 <li>Always test that <path>package.mask</path> is okay by doing 
 <c>emerge --pretend mypkg</c> before you commit and check 
 that it doesn't contain any conflicts.</li>
-<li>Always update the <path>ChangeLog</path> before you commit.</li>
-<li>Always commit the updated <path>package.mask</path> before 
-the updated package, in case conflicts occur while you commit 
-<path>package.mask</path>.</li>
+<li>Always commit the updated <path>package.mask</path> before
+the updated package.</li>
 <li>Always do atomic commits; if you commit a package with a new license, 
 or that is masked, then first commit the revised <path>package.mask</path> and/or license, 
 then commit the ebuild, <path>ChangeLog</path>, patches
 and <uri link="::ebuild-writing/misc-files/metadata">metadata.xml</uri> all in <b>one</b> go
-to avoid breaking users' installations.</li>
+.</li>
+<note> Although the set of changes in a single git commit is atomic, and
+combining <path>package.mask</path>/license changes with ebuild changes in a
+single commit wouldn't break atomicity, it is not currently possible to do so
+using <c>repoman commit</c>.</note>
+<!-- See: https://bugs.gentoo.org/show_bug.cgi?id=390651 -->
 </ul>
 
 </body>
-- 
2.4.10

[gentoo-dev] [PATCH 10/16] ebuild-maintenance: rewrite the subsection on upgrading ebuilds for git #558642

[Gokturk Yuksek]

Substitute the mentions of CVS with git.
Rewrite CVS commands with their git equivalents.
Delete the reference to editing ChangeLog.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-maintenance/text.xml b/ebuild-maintenance/text.xml
index 2d46749..a1d92a1 100644
--- a/ebuild-maintenance/text.xml
+++ b/ebuild-maintenance/text.xml
@@ -275,14 +275,14 @@ available.
 
 <p>
 If there are no new dependencies, do not remove keywords if your
-commit fails with repoman - please try a full <c>cvs update</c> and if
+commit fails with repoman - please try a full <c>git pull</c> and if
 you still have problems, then commit with <c>repoman -I</c> and file a
-bug to the broken architecture, noting it in your CVS commit message.
+bug to the broken architecture, noting it in your git commit message.
 </p>
 
 <warn>
 When committing, make sure that you reference any bugs in the
-ChangeLog as well as the CVS message. Failing to do so is considered
+commit message. Failing to do so is considered
 to be in very poor taste and may result in disciplinary action.
 </warn>
 
-- 
2.4.10

[gentoo-dev] [PATCH 11/16] ebuild-maintenance: rewrite the subsection on moving ebuilds for git #558642

[Gokturk Yuksek]

Substitute the mentions of CVS with git.
Rewrite CVS commands with their git equivalents.
Remove the paragraph about deleting directories in CVS.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-maintenance/text.xml b/ebuild-maintenance/text.xml
index a1d92a1..8c542d0 100644
--- a/ebuild-maintenance/text.xml
+++ b/ebuild-maintenance/text.xml
@@ -298,7 +298,7 @@ Moving ebuilds is a two-step process:
 </p>
 
 <p>
-Firstly, you need to move the ebuild in CVS. To do this, you should
+Firstly, you need to move the ebuild in the git tree. To do this, you should
 copy the ebuild to its new location and commit that as you would with
 a <uri link="#adding-a-new-ebuild">new ebuild</uri>.
 </p>
@@ -323,23 +323,17 @@ for <path>net-firewall/fwbuilder</path> when they are available.
 
 <p>
 Once this step is concluded, you are allowed to remove the old package.
-Simply issue a <c>cvs remove -Rf $PN</c> in the package category and commit
+Simply issue a <c>git rm -rf $PN</c> in the package category and commit
 the changes afterwards with a meaningful commit message. Don't forget to update
 entries in files such as profiles/package.mask to reflect the new category. Finally
 remember to change the title to open bugs related to this package if needed.
 </p>
 
 <pre caption="Removing a package">
-net-misc # cvs rm -Rf fwbuilder
-cvs remove: use `cvs commit' to remove these files permanently
-net-misc # cvs ci -m "Moving net-misc/fwbuilder to net-firewall/fwbuilder."
+net-misc # git rm -rf fwbuilder
+net-misc # git commit --gpg-sign -m "Moving net-misc/fwbuilder to net-firewall/fwbuilder."
 </pre>
 
-<note>
-CVS cannot destroy directories: it will simply not re-create them if
-they are blank, providing you use CVS with the <c>-P</c> flag.
-</note>
-
 </body>
 </section>
 
-- 
2.4.10

[gentoo-dev] [PATCH 12/16] ebuild-maintenance: rewrite the subsection on removing ebuilds for git #558642

[Gokturk Yuksek]

Substitute the mentions of CVS with git.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-maintenance/text.xml b/ebuild-maintenance/text.xml
index 8c542d0..4078d46 100644
--- a/ebuild-maintenance/text.xml
+++ b/ebuild-maintenance/text.xml
@@ -364,8 +364,8 @@ entry which may be affected by your change.
 
 <p>
 When removing an ebuild make sure that no dependencies in Portage are broken
-due to the removal - additionally, your CVS commit message should explain
-clearly why the ebuild is being removed from CVS.
+due to the removal - additionally, your git commit message should explain
+clearly why the ebuild is being removed from the git repository.
 </p>
 
 <p>
-- 
2.4.10

[gentoo-dev] [PATCH 13/16] ebuild-maintenance: rewrite the subsection on removing packages for git #558642

[Gokturk Yuksek]

Substitute the mentions of CVS with git.
Rewrite CVS commands with their git equivalents.
Remove the paragraph about deleting directories in CVS.

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-maintenance/text.xml b/ebuild-maintenance/text.xml
index 4078d46..7014689 100644
--- a/ebuild-maintenance/text.xml
+++ b/ebuild-maintenance/text.xml
@@ -396,20 +396,15 @@ When removing packages follow these steps:
   <li>Send last rites to gentoo-dev-announce and gentoo-dev</li>
   <li>Mask the package</li>
   <li>Wait 30 days (or more)</li>
-  <li>Remove from CVS unless the reason for removal has been fixed</li>
+  <li>Remove from the git tree unless the reason for removal has been fixed</li>
   <li>Remove package.mask entry</li>
   <li>Close open bugs as WONTFIX</li>
 </ol> 
 
-<p>
-In order to remove a package completely from CVS, delete any files from the
-directory and commit this, CVS will take care of removing empty directories
-itself.
-</p>
-
-<pre caption="Removing a package from CVS"><comment>#</comment> <keyword>cd</keyword> app-admin
-<comment>#</comment> <keyword>cvs</keyword> rm -Rf scotty
-<comment>#</comment> <keyword>cvs</keyword> ci -m "app-admin/scotty removal (pending 21st July 2006), see #77501 for reference." scotty</pre>
+<pre caption="Removing a package from git">
+<comment>#</comment> <keyword>cd</keyword> app-admin
+<comment>#</comment> <keyword>git</keyword> rm -rf scotty
+<comment>#</comment> <keyword>git</keyword> commit --gpg-sign -m "app-admin/scotty: removal (pending 21st July 2006), see #77501 for reference." scotty</pre>
 
 </body>
 </section>
-- 
2.4.10

[gentoo-dev] [PATCH 14/16] ebuild-maintenance: replace "cvs commit" with "git commit" #558642

[Gokturk Yuksek]

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/ebuild-maintenance/text.xml b/ebuild-maintenance/text.xml
index 7014689..6dc8813 100644
--- a/ebuild-maintenance/text.xml
+++ b/ebuild-maintenance/text.xml
@@ -62,8 +62,8 @@ examples.
 </p>
 
 <p>
-When committing to CVS, all developers should use <c>repoman commit</c> 
-instead of <c>cvs commit</c> to submit their ebuilds.  Before committing, 
+When committing to git, all developers should use <c>repoman commit</c>
+instead of <c>git commit</c> to submit their ebuilds.  Before committing,
 please run <c>repoman full</c> to make sure you didn't forget something.
 </p>
 
-- 
2.4.10

[gentoo-dev] [PATCH 15/16] general-concepts/git-to-rsync/diagram: update the description for git #558642

[Gokturk Yuksek]

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/general-concepts/git-to-rsync/diagram.svg b/general-concepts/git-to-rsync/diagram.svg
index 8cf7414..2ee9608 100644
--- a/general-concepts/git-to-rsync/diagram.svg
+++ b/general-concepts/git-to-rsync/diagram.svg
@@ -2,7 +2,7 @@
 <!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN"
     "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
 <svg width="700" height="250" xmlns="http://www.w3.org/2000/svg" version="1.1">
-    <desc>CVS to RSYNC Propagation</desc>
+    <desc>Git to RSYNC Propagation</desc>
 
     <rect x="-10" y="-10" width="1000" height="1000" fill="#eeeeee" id="background" />
 
-- 
2.4.10

[gentoo-dev] [PATCH 16/16] general-concepts/git-to-rsync/diagram: update the description for git #558642

[Gokturk Yuksek]

X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>

diff --git a/general-concepts/mirrors/diagram.svg b/general-concepts/mirrors/diagram.svg
index 0af1f1c..b807e5a 100644
--- a/general-concepts/mirrors/diagram.svg
+++ b/general-concepts/mirrors/diagram.svg
@@ -52,7 +52,7 @@
   <defs
      id="defs1680" />
   <desc
-     id="desc1548">CVS to RSYNC Propagation</desc>
+     id="desc1548">Git to RSYNC Propagation</desc>
   <rect
      y="0.24284256"
      x="-2.0556145"
-- 
2.4.10

[gentoo-dev] Re: [PATCH 05/16] ebuild-writing/misc-files: replace the code for cvs commit with git #558642

[Michael Palimaka]

On 01/17/2016 06:55 PM, Gokturk Yuksek wrote:
> Replace "cvs commit" with the equivalent "git add && git commit" version.
> 
> X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
> Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>
> 
> diff --git a/ebuild-writing/misc-files/metadata/text.xml b/ebuild-writing/misc-files/metadata/text.xml
> index e506b1c..a8beacc 100644
> --- a/ebuild-writing/misc-files/metadata/text.xml
> +++ b/ebuild-writing/misc-files/metadata/text.xml
> @@ -532,7 +532,8 @@ is currently:
>  <pre>
>  xmllint --noout --valid metadata.xml
>  glep31check metadata.xml
> -cvs commit -m "Adding category metadata.xml for my-category" metadata.xml
> +git add metadata.xml
> +git commit --gpg-sign -m "Adding category metadata.xml for my-category"
>  </pre>
>  
>  </body>
> 

I would suggest a commit message of something like: "my-category: adding
category metadata.xml".

[gentoo-dev] Re: [PATCH 05/16] ebuild-writing/misc-files: replace the code for cvs commit with git #558642

[Michael Palimaka]

On 01/17/2016 06:55 PM, Gokturk Yuksek wrote:
> Replace "cvs commit" with the equivalent "git add && git commit" version.
> 
> X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
> Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>
> 
> diff --git a/ebuild-writing/misc-files/metadata/text.xml b/ebuild-writing/misc-files/metadata/text.xml
> index e506b1c..a8beacc 100644
> --- a/ebuild-writing/misc-files/metadata/text.xml
> +++ b/ebuild-writing/misc-files/metadata/text.xml
> @@ -532,7 +532,8 @@ is currently:
>  <pre>
>  xmllint --noout --valid metadata.xml
>  glep31check metadata.xml
> -cvs commit -m "Adding category metadata.xml for my-category" metadata.xml
> +git add metadata.xml
> +git commit --gpg-sign -m "Adding category metadata.xml for my-category"
>  </pre>
>  
>  </body>
> 

I would suggest a commit message of something like: "my-category: adding
category metadata.xml".

[gentoo-dev] Re: [PATCH 11/16] ebuild-maintenance: rewrite the subsection on moving ebuilds for git #558642

[Michael Palimaka]

On 01/17/2016 06:55 PM, Gokturk Yuksek wrote:
> Substitute the mentions of CVS with git.
> Rewrite CVS commands with their git equivalents.
> Remove the paragraph about deleting directories in CVS.
> 
> X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
> Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>
> 
> diff --git a/ebuild-maintenance/text.xml b/ebuild-maintenance/text.xml
> index a1d92a1..8c542d0 100644
> --- a/ebuild-maintenance/text.xml
> +++ b/ebuild-maintenance/text.xml
> @@ -298,7 +298,7 @@ Moving ebuilds is a two-step process:
>  </p>
>  
>  <p>
> -Firstly, you need to move the ebuild in CVS. To do this, you should
> +Firstly, you need to move the ebuild in the git tree. To do this, you should
>  copy the ebuild to its new location and commit that as you would with
>  a <uri link="#adding-a-new-ebuild">new ebuild</uri>.
>  </p>
> @@ -323,23 +323,17 @@ for <path>net-firewall/fwbuilder</path> when they are available.
>  
>  <p>
>  Once this step is concluded, you are allowed to remove the old package.
> -Simply issue a <c>cvs remove -Rf $PN</c> in the package category and commit
> +Simply issue a <c>git rm -rf $PN</c> in the package category and commit
>  the changes afterwards with a meaningful commit message. Don't forget to update
>  entries in files such as profiles/package.mask to reflect the new category. Finally
>  remember to change the title to open bugs related to this package if needed.
>  </p>
>  
>  <pre caption="Removing a package">
> -net-misc # cvs rm -Rf fwbuilder
> -cvs remove: use `cvs commit' to remove these files permanently
> -net-misc # cvs ci -m "Moving net-misc/fwbuilder to net-firewall/fwbuilder."
> +net-misc # git rm -rf fwbuilder
> +net-misc # git commit --gpg-sign -m "Moving net-misc/fwbuilder to net-firewall/fwbuilder."
>  </pre>
>  
> -<note>
> -CVS cannot destroy directories: it will simply not re-create them if
> -they are blank, providing you use CVS with the <c>-P</c> flag.
> -</note>
> -
>  </body>
>  </section>
>  
> 

How about git mv instead?

Re: [gentoo-dev] [PATCH 05/16] ebuild-writing/misc-files: replace the code for cvs commit with git #558642

[Manuel Rüger]

[-- Attachment #1: Type: text/plain, Size: 925 bytes --]

On 17.01.2016 08:55, Gokturk Yuksek wrote:
> Replace "cvs commit" with the equivalent "git add && git commit" version.
> 
> X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642
> Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>
> 
> diff --git a/ebuild-writing/misc-files/metadata/text.xml b/ebuild-writing/misc-files/metadata/text.xml
> index e506b1c..a8beacc 100644
> --- a/ebuild-writing/misc-files/metadata/text.xml
> +++ b/ebuild-writing/misc-files/metadata/text.xml
> @@ -532,7 +532,8 @@ is currently:
>  <pre>
>  xmllint --noout --valid metadata.xml
>  glep31check metadata.xml
> -cvs commit -m "Adding category metadata.xml for my-category" metadata.xml
> +git add metadata.xml
> +git commit --gpg-sign -m "Adding category metadata.xml for my-category"
>  </pre>
>  
>  </body>
> 

Can we discourage to use "-m" and prefer to open up an editor instead?

Cheers,

Manuel


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 605 bytes --]

Re: [gentoo-dev] [PATCH 00/16] devmanual: update the docs for post git-migration #558642

[Michael Orlitzky]

On 01/17/2016 02:55 AM, Gokturk Yuksek wrote:
> Hi,
> 
> The following patch series attempt to update devmanual to replace the instances
> of CVS with their git equivalents.

Thank you!

> - I am not sure about what to do with the subsection
>   Ebuild Functions/Miscellaneous Files/ChangeLog. The ChangeLog files
>   still exist in the rsync tree, however they are of little importance
>   in terms of development.
> 

I should probably update the summary, but I looked at this as part of,

  https://bugs.gentoo.org/show_bug.cgi?id=485314

and decided to get rid of it. Any mention of echangelog can go, too.

Re: [gentoo-dev] [PATCH 00/16] devmanual: update the docs for post git-migration #558642

[Justin Lecher (jlec)]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

On 17/01/16 08:55, Gokturk Yuksek wrote:
> Hi,
> 
> The following patch series attempt to update devmanual to replace
> the instances of CVS with their git equivalents. I tried to keep
> the changes small in favor of updating the docs over multiple
> iterations of patch series. The intention is to get the bare bone
> changes in so that the quiz questions can be answered.
> 
> Currently not included in the series are: - References to CVS in
> sections about herds since they will have to be rewritten per GLEP
> 67 soon anyway. See:
> https://bugs.gentoo.org/show_bug.cgi?id=572144 - I am not sure
> about what to do with the subsection Ebuild Functions/Miscellaneous
> Files/ChangeLog. The ChangeLog files still exist in the rsync tree,
> however they are of little importance in terms of development.
> 

Thanks a bunch for all your work.

Please direct the mail to devmanual@gentoo.org. That alias takes care
of the devmanual changes.

Justin

-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0

iQJ8BAEBCgBmBQJWm7CRXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w
ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQ0QUU0N0I4NzFERUI0MTJFN0EyODE0NUFF
OTQwMkE3OUIwMzUyOUEyAAoJEOlAKnmwNSmiIMwP/2/9RGTZYlPjjmmc8kao77kg
C0ogMmWjsazNSdUcMztqwyDNMPMtCf1ZbVLq/MJYxFkKH6lXocP6/MDoOgIrACCU
gCQhOASG/8fVyLAyT/719qsXPZpEXAqKAImHkB+jvw4SG0W7+uORQWUGXAKd5Oic
ZAfWhqBjCkk7pWlP8u+UIM1E8fb3e1KPednbAetNCdRQRiCh77X5BJODAMc25T3O
vu5qMxB2UDp4tjZ2+rKahcHabgIgp4vbkNk91t5tskOu9aiQCDCkVyBbsUVlfWd+
PHMxfXI+p19K8eIv7tocdX3bKVdyCtaaB6bBddb9G0skM1Kpi5PK0W2+2H/jBEvL
YfQ0gNAENhPK5aAPGR9Z1ulN7aSaiWMEExfF6rTx+5BF+IKzcH4n4jKsE2AIw25c
hJC0DlugAW7IXUi7QKCSpbejw+BhjtWppo5f8bgGK+pUot6rbKmItfgcCFOtyZjV
oFKL5ZiGpclPaYw8bBh9woFypc1K1cHCkDXrFbag1BXTugit2f0qK2716uL45UcT
xqGp1uJ/lm4dS91sptfjOCFVs76df8xl8jzr3zEwmfgac0bjCR7MVucTBvHmP06A
UCtNNvKI6GRfjC1BtZ+WpW+Mg6wBcdIu8I46r9wdciVbwqJ8KHAoQ0/o2RRVLYbP
Uv/ioCuMAUpTAFbUhPVt
=p90x
-----END PGP SIGNATURE-----

Re: [gentoo-dev] Re: [PATCH 11/16] ebuild-maintenance: rewrite the subsection on moving ebuilds for git #558642

[Göktürk Yüksek]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Michael Palimaka:
> On 01/17/2016 06:55 PM, Gokturk Yuksek wrote:
>> Substitute the mentions of CVS with git. Rewrite CVS commands
>> with their git equivalents. Remove the paragraph about deleting
>> directories in CVS.
>> 
>> X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642 
>> Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>
>> 
>> diff --git a/ebuild-maintenance/text.xml
>> b/ebuild-maintenance/text.xml index a1d92a1..8c542d0 100644 ---
>> a/ebuild-maintenance/text.xml +++ b/ebuild-maintenance/text.xml 
>> @@ -298,7 +298,7 @@ Moving ebuilds is a two-step process: </p>
>> 
>> <p> -Firstly, you need to move the ebuild in CVS. To do this, you
>> should +Firstly, you need to move the ebuild in the git tree. To
>> do this, you should copy the ebuild to its new location and
>> commit that as you would with a <uri
>> link="#adding-a-new-ebuild">new ebuild</uri>. </p> @@ -323,23
>> +323,17 @@ for <path>net-firewall/fwbuilder</path> when they are
>> available.
>> 
>> <p> Once this step is concluded, you are allowed to remove the
>> old package. -Simply issue a <c>cvs remove -Rf $PN</c> in the
>> package category and commit +Simply issue a <c>git rm -rf $PN</c>
>> in the package category and commit the changes afterwards with a
>> meaningful commit message. Don't forget to update entries in
>> files such as profiles/package.mask to reflect the new category.
>> Finally remember to change the title to open bugs related to this
>> package if needed. </p>
>> 
>> <pre caption="Removing a package"> -net-misc # cvs rm -Rf
>> fwbuilder -cvs remove: use `cvs commit' to remove these files
>> permanently -net-misc # cvs ci -m "Moving net-misc/fwbuilder to
>> net-firewall/fwbuilder." +net-misc # git rm -rf fwbuilder 
>> +net-misc # git commit --gpg-sign -m "Moving net-misc/fwbuilder
>> to net-firewall/fwbuilder." </pre>
>> 
>> -<note> -CVS cannot destroy directories: it will simply not
>> re-create them if -they are blank, providing you use CVS with the
>> <c>-P</c> flag. -</note> - </body> </section>
>> 
>> 
> 
> How about git mv instead?
> 
> 
The workflow in the devmanual is: copy-mask-remove.
Git mv workflow is similar to: copy-remove-mask.
If encapsulated in a single commit, the atomicity is preserved and it
shouldn't violate anything. I actually wrote it using git mv first and
rewrote to match the current workflow of devmanual.

If there is no objection to this, I'll rewrite it with git mv.

- --
gokturk

-----BEGIN PGP SIGNATURE-----

iQEcBAEBCgAGBQJWnBBbAAoJEIT4AuXAiM4zD0YIAM3oUGd5woCtq4/Xou5+R86D
6ofPQQTsjZ7C/GE9dImmch0Mn1iG2Pt8kxP1G/M9br0Vs8D7qPQFO+5Xmm+SBhO7
8s0KSwYnipWqzEKuDtbNwBXJ2mj9eG/0NQKdOXbxKWWk/QMewy+DSqZekiQaL512
JmacEPm/TzAukYQqpwx8rGw3kp+Nz7v+TTQ47epElQ/wnRPZyU05JPyeYFbsV97y
aG17JIRx+FnL6rOKJJE/pyZ3Bbqu1autkgw0cL9Wk5EdxVm3WeR/nHj847EcziTd
Ws312So+GEhb2smFdLxXBN6y/tiTb1xeeNds2sMsSu2KNfNN3E7VWOXSsjK8mMI=
=SuBZ
-----END PGP SIGNATURE-----

[gentoo-dev] Re: [PATCH 05/16] ebuild-writing/misc-files: replace the code for cvs commit with git #558642

[Göktürk Yüksek]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Michael Palimaka:
> On 01/17/2016 06:55 PM, Gokturk Yuksek wrote:
>> Replace "cvs commit" with the equivalent "git add && git commit"
>> version.
>> 
>> X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642 
>> Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>
>> 
>> diff --git a/ebuild-writing/misc-files/metadata/text.xml
>> b/ebuild-writing/misc-files/metadata/text.xml index
>> e506b1c..a8beacc 100644 ---
>> a/ebuild-writing/misc-files/metadata/text.xml +++
>> b/ebuild-writing/misc-files/metadata/text.xml @@ -532,7 +532,8 @@
>> is currently: <pre> xmllint --noout --valid metadata.xml 
>> glep31check metadata.xml -cvs commit -m "Adding category
>> metadata.xml for my-category" metadata.xml +git add metadata.xml 
>> +git commit --gpg-sign -m "Adding category metadata.xml for
>> my-category" </pre>
>> 
>> </body>
>> 
> 
> I would suggest a commit message of something like: "my-category:
> adding category metadata.xml".
> 
I have seen its use in the git logs. If nobody has objections, I'll
include it in that format for v2 of the patch series.

- --
gokturk

-----BEGIN PGP SIGNATURE-----

iQEcBAEBCgAGBQJWnBIHAAoJEIT4AuXAiM4zsgQH/30nRVmRzOnzXwXgpRlQcvcC
kiFy3uAcYApzl69rDkK3ybz2MG6K3W5PCQjyzUXn99WeNefiysr+5ufPQka65Ytt
QKQ8RfIOXOVW8t4rvQF/G018NMAUKq6qk9gEwvrs9SkuJ93x/r8D20d78Q5ZPALi
OaKlIpmj4HUZ1nG+MeY+wazTqMDZrs/ArWVv/q4UGGa/M3IhG2qN8cClv3h/rdKJ
nxl4+QGsDQlVJjw1hFPF0NXPq8LQhg2OluVuYy0DnZ70FGTqIOodnB/znaVGZI6o
z7A/FV3RVV03wLYS6zqo5c/yPmcatGDo2Rk/4rLybG41qqy9NKZXLjZhk+L9loo=
=olHD
-----END PGP SIGNATURE-----

Re: [gentoo-dev] [PATCH 05/16] ebuild-writing/misc-files: replace the code for cvs commit with git #558642

[Göktürk Yüksek]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Manuel Rüger:
> On 17.01.2016 08:55, Gokturk Yuksek wrote:
>> Replace "cvs commit" with the equivalent "git add && git commit"
>> version.
>> 
>> X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=558642 
>> Signed-off-by: Gokturk Yuksek <gokturk@binghamton.edu>
>> 
>> diff --git a/ebuild-writing/misc-files/metadata/text.xml
>> b/ebuild-writing/misc-files/metadata/text.xml index
>> e506b1c..a8beacc 100644 ---
>> a/ebuild-writing/misc-files/metadata/text.xml +++
>> b/ebuild-writing/misc-files/metadata/text.xml @@ -532,7 +532,8 @@
>> is currently: <pre> xmllint --noout --valid metadata.xml 
>> glep31check metadata.xml -cvs commit -m "Adding category
>> metadata.xml for my-category" metadata.xml +git add metadata.xml 
>> +git commit --gpg-sign -m "Adding category metadata.xml for
>> my-category" </pre>
>> 
>> </body>
>> 
> 
> Can we discourage to use "-m" and prefer to open up an editor
> instead?
> 
I think the purpose of "-m" is only to illustrate the example of
adding a category with a proper message rather than suggesting a
workflow. Note that the section for committing packages suggests
"repoman commit" which opens up an editor by default.

Is there a particular reason why we should discourage the use of "-m"?

> Cheers,
> 
> Manuel
> 

- --
gokturk

-----BEGIN PGP SIGNATURE-----

iQEcBAEBCgAGBQJWnBQ3AAoJEIT4AuXAiM4zkeUH/AuswPQV3Fw+aeuI08l8xjpx
q+BVZ8c7/5RNqgsQGF/0uKnkZI70WwbkLQwsk2nlHKXjuL5qUWmQ16YBC+fF03SG
JD5eiFhqnzUK79Y5i2Rnfoc2o6DlcLzBSXHXMjtVldg4ufsq+v7mP+Jw901MyJu+
VBY6gkjmkqsahXeaPqmuoXslcr3yQtDHKVU4m9YOptZ/0jTDg4asnvhpv6ERwdNL
uSrO4hzdbh314/o/R/4TWhGV56wlRVUaelzaVwDaBt2wxY5gYyKWhRizN5Exu8/w
BlWSDqwfpKbvHMS2ArrDh0PvvWqEZAMhNuXX2hateO2wQIJlrWZ5KSId5ioW0fw=
=s7Dj
-----END PGP SIGNATURE-----

Re: [gentoo-dev] [PATCH 00/16] devmanual: update the docs for post git-migration #558642

[Göktürk Yüksek]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Michael Orlitzky:
> On 01/17/2016 02:55 AM, Gokturk Yuksek wrote:
>> Hi,
>> 
>> The following patch series attempt to update devmanual to replace
>> the instances of CVS with their git equivalents.
> 
> Thank you!
> 
>> - I am not sure about what to do with the subsection Ebuild
>> Functions/Miscellaneous Files/ChangeLog. The ChangeLog files 
>> still exist in the rsync tree, however they are of little
>> importance in terms of development.
>> 
> 
> I should probably update the summary, but I looked at this as part
> of,
> 
> https://bugs.gentoo.org/show_bug.cgi?id=485314
> 
> and decided to get rid of it. Any mention of echangelog can go,
> too.
> 
> 
Thanks for pointing this out to me. I will include your patches in the
v2 of the patchset and remove ChangeLog and echangelog unless there
are objections.

- --
gokturk

-----BEGIN PGP SIGNATURE-----

iQEcBAEBCgAGBQJWnBWXAAoJEIT4AuXAiM4zY2YH/RfE1deC6cHJy1TDUu4LG6WW
1MmrKS6PO7TJ5YNDENToprqg2n/YaP4yAf7jd6KOFDZ/KnXoal2h0L6pUW55TDCZ
TsanPjNLWkkICfZ33msB5jbqYU6jrcoQhqQK7to1J9CgHfQyVcBlDrxpHzYnMFO7
8nUZ1LTluroN1bXCiKljN41c6gFYM02hMep97RcxYR3KEiQSuiC9Ja4UA1SYHnjq
jSmgbWo31IJUUV3K9VSIM4ZzcbNyB5U+rBKmoH7V/UzLqnW8AJWLJ21mAcHwZkSA
iiuWuxqyltLZb/TXibRDnBu2cr8RXG6jx2EIzd3wu1QNwGWRbX6wfWLKFNmgKCI=
=5LoC
-----END PGP SIGNATURE-----

Re: [gentoo-dev] [PATCH 00/16] devmanual: update the docs for post git-migration #558642

[Göktürk Yüksek]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Justin Lecher (jlec):
> On 17/01/16 08:55, Gokturk Yuksek wrote:
>> Hi,
> 
>> The following patch series attempt to update devmanual to
>> replace the instances of CVS with their git equivalents. I tried
>> to keep the changes small in favor of updating the docs over
>> multiple iterations of patch series. The intention is to get the
>> bare bone changes in so that the quiz questions can be answered.
> 
>> Currently not included in the series are: - References to CVS in 
>> sections about herds since they will have to be rewritten per
>> GLEP 67 soon anyway. See: 
>> https://bugs.gentoo.org/show_bug.cgi?id=572144 - I am not sure 
>> about what to do with the subsection Ebuild
>> Functions/Miscellaneous Files/ChangeLog. The ChangeLog files
>> still exist in the rsync tree, however they are of little
>> importance in terms of development.
> 
> 
> Thanks a bunch for all your work.
> 
> Please direct the mail to devmanual@gentoo.org. That alias takes
> care of the devmanual changes.
> 
I accidentally assumed it belongs to gentoo-doc, thanks for the heads
up. In the following weeks, I would like to follow up with GLEP 67 and
EAPI6 patches for devmanual and I can benefit from the review on
gentoo-dev. Is it OK to send the email to devmanual@g.o and CC
gentoo-dev@lists.g.o?

> Justin
> 
> 

- --
gokturk
-----BEGIN PGP SIGNATURE-----

iQEcBAEBCgAGBQJWnBZUAAoJEIT4AuXAiM4zoBwIAMzOyqoykAUJhHIoQGY7IoqF
bAys2BdJwv4UKL+Umt35IYgJDCN8GMklkwMToJ6flzRWMzcwCr2NN0yVXTSln3vz
1tdGxtv2w8afsbVgeE9eL7ZaBz9Zp/sdBwvMV6VMptC1esrLL2CqK7nx4s8x76UU
k1B6cyJBQKjyFB843fP7Z4NNIVHELkpGVf7RXD2SUeS0gtnpxveqwR0GghYYfSI1
RTXNtVTwbve97C3ewxv4hGNs8WKEoWdt8pNDBU7YT+CwIB6+YiGJcEDeogL6SK/7
TpckdS6i329o9gxWLJlt4tz+DjNUErb0TMJVTVBnla/6ohDHEIhW0za/46M9O8E=
=QGFi
-----END PGP SIGNATURE-----

Re: [gentoo-dev] [PATCH 05/16] ebuild-writing/misc-files: replace the code for cvs commit with git #558642

[NP-Hardass]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Several reasons for discouraging -m.  Here are two (there may be others)
:
Firstly, with repoman commit, -m prepends "category/pkgname:" which is
the style required as per the git workflow wiki.
Additionally, for both repoman commit and git commit opening in an
editor allows the user to make sure the first line is under 80 chars
and enables the user to more easily conform to the desired format by
having additional information on lines >=3

On 01/17/2016 05:22 PM, Göktürk Yüksek wrote:
> Manuel Rüger:
>> On 17.01.2016 08:55, Gokturk Yuksek wrote:
>>> Replace "cvs commit" with the equivalent "git add && git
>>> commit" version.
>>> 
>>> X-Gentoo-Bug-URL:
>>> https://bugs.gentoo.org/show_bug.cgi?id=558642 Signed-off-by:
>>> Gokturk Yuksek <gokturk@binghamton.edu>
>>> 
>>> diff --git a/ebuild-writing/misc-files/metadata/text.xml 
>>> b/ebuild-writing/misc-files/metadata/text.xml index 
>>> e506b1c..a8beacc 100644 --- 
>>> a/ebuild-writing/misc-files/metadata/text.xml +++ 
>>> b/ebuild-writing/misc-files/metadata/text.xml @@ -532,7 +532,8
>>> @@ is currently: <pre> xmllint --noout --valid metadata.xml 
>>> glep31check metadata.xml -cvs commit -m "Adding category 
>>> metadata.xml for my-category" metadata.xml +git add
>>> metadata.xml +git commit --gpg-sign -m "Adding category
>>> metadata.xml for my-category" </pre>
>>> 
>>> </body>
>>> 
> 
>> Can we discourage to use "-m" and prefer to open up an editor 
>> instead?
> 
> I think the purpose of "-m" is only to illustrate the example of 
> adding a category with a proper message rather than suggesting a 
> workflow. Note that the section for committing packages suggests 
> "repoman commit" which opens up an editor by default.
> 
> Is there a particular reason why we should discourage the use of
> "-m"?
> 
>> Cheers,
> 
>> Manuel
> 
> 
> 

- -- 
NP-Hardass
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCAAGBQJWnCFkAAoJEBzZQR2yrxj7p54P/2gBHfvyEJx59VrMSUZm/ZZi
fzQcOeOjRHWtI1Uz1f15J2eVjq0qQ+B8Wddczsq9Xj70sb8Bgd42B87Im+qo8AmD
2DJVkAveOEOd8UTz6BOkRqksYc2EDHLMv9Nu4ZukAdM56NwNQEfE2du9N/sOVr1D
LAuni26JD2nhIGc749qwUX46gijGEww5gOwNEtMlSS59XeLuYnWSDacxKj19zApM
Zx8HfHlysILT8EkM0HZ73fKEcAfT7ZKbgq2rKlPv4FwogprNcj/IWasD5o9d2d7b
0zFLExdBMGVHGHDc1hu8M8ofFB0B0cRa+3QEJX3hvSxPOqmRtb8E3GAu20JtPJ69
UVivK7CT1qQIaLuIlGC95Ny06ZRLKp5mNQ1Del6pDDJHjs3b2O3P1wTTNH/RuE5V
MZGKQpLojpmNTepOiBbOxfKJjvDhoQ5yzJ7qw0q4K23awRPj5j/5plKt8kwznySH
EBNLqRl8E+Q1BJ91wNJZVtXtOhc0Lu+03cEJ95D1f55sOUdStnjBaMiZAKP3gswa
YWdaRsONtuCx6BEhILdd0lVOnCIz75YwW8Le3DyitLO3Klf9WA2EeYAYmWmiLpas
L6Z7SU1I0M//gJFq1JdeEAZGNoVoVQJBVQ04tM+9VMxijmGdArzdRmye41J+60ZR
ctBkIkyPn5mlmrOp/Qxo
=pg/V
-----END PGP SIGNATURE-----

[gentoo-dev] Re: [PATCH 00/16] devmanual: update the docs for post git-migration #558642

[Duncan]

Göktürk Yüksek posted on Sun, 17 Jan 2016 17:31:54 -0500 as excerpted:

> Justin Lecher (jlec):
>> On 17/01/16 08:55, Gokturk Yuksek wrote:
>> 
>>> The following patch series attempt to update devmanual to replace the
>>> instances of CVS with their git equivalents.
>> 
>> Please direct the mail to devmanual@gentoo.org. That alias takes care
>> of the devmanual changes.
>> 
> In the following weeks, I would like to follow up with GLEP 67 and
> EAPI6 patches for devmanual and I can benefit from the review on
> gentoo-dev. Is it OK to send the email to devmanual@g.o and CC
> gentoo-dev@lists.g.o?

While I'm not a dev and thus can't give the OK, I'd say "Yes, please."

I think a lot more people see it and can benefit when it's on gentoo-
dev.  I know I'm finding it interesting, even simply as a long-time
gentoo user and follower of the gentoo-dev. =:^)

-- 
Duncan - List replies preferred.   No HTML msgs.
"Every nonfree program has a lord, a master --
and if you use the program, he is your master."  Richard Stallman