Change in vdsm[master]: tool: fill AUTHOR at vdsm-tool manual page.

2014-09-29 Thread mtayer
mooli tayer has posted comments on this change.

Change subject: tool: fill AUTHOR at vdsm-tool manual page.
..


Patch Set 2: Verified+1

If There is agreement I will add the author as well.

AUTHOR
   This manual page was written by Andrew Dahms for the oVirt project.

-- 
To view, visit http://gerrit.ovirt.org/33455
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I73a0fadec63c6643ae8d3678b611bbf15710142a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer mta...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: mooli tayer mta...@redhat.com
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: tool: fill AUTHOR at vdsm-tool manual page.

2014-09-29 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: tool: fill AUTHOR at vdsm-tool manual page.
..


Patch Set 1:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11690/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/12634/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/12479/ : SUCCESS

-- 
To view, visit http://gerrit.ovirt.org/33455
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I73a0fadec63c6643ae8d3678b611bbf15710142a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer mta...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: mooli tayer mta...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: tool: fill AUTHOR at vdsm-tool manual page.

2014-09-29 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: tool: fill AUTHOR at vdsm-tool manual page.
..


Patch Set 2:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11691/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/12635/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/12480/ : SUCCESS

-- 
To view, visit http://gerrit.ovirt.org/33455
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I73a0fadec63c6643ae8d3678b611bbf15710142a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer mta...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: mooli tayer mta...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: tool: fill AUTHOR at vdsm-tool manual page.

2014-09-29 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: tool: fill AUTHOR at vdsm-tool manual page.
..


Patch Set 2:

Looks ok, but note that I know nothing about man formatting :-)

-- 
To view, visit http://gerrit.ovirt.org/33455
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I73a0fadec63c6643ae8d3678b611bbf15710142a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer mta...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: mooli tayer mta...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: tool: fill AUTHOR at vdsm-tool manual page.

2014-09-29 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: tool: fill AUTHOR at vdsm-tool manual page.
..


Patch Set 2:

Can we simply drop the AUTHORS line?

according to man-pages(7):

   AUTHORS   lists authors of the documentation or program.  Use of an 
AUTHORS
 section is strongly discouraged.  Generally, it is better 
not  to
 clutter  every  page with a list of (over time potentially 
numer‐
 ous) authors; if you write or significantly amend a page,  
add  a
 copyright notice as a comment in the source file.  If you 
are the
 author of a device driver and want  to  include  an  
address  for
 reporting bugs, place this under the BUGS section.

-- 
To view, visit http://gerrit.ovirt.org/33455
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I73a0fadec63c6643ae8d3678b611bbf15710142a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer mta...@redhat.com
Gerrit-Reviewer: Andrew Dahms andrewjda...@gmail.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Nir Soffer nsof...@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim ybron...@redhat.com
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: mooli tayer mta...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches