Re: [PATCH V5 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On a Monday in 2023, Daniel P. Berrangé wrote:
On Mon, Jan 09, 2023 at 09:03:44AM -0800, Andrea Bolognani wrote:
On Mon, Jan 09, 2023 at 09:49:04AM -0700, Jim Fehlig wrote:
> On 1/9/23 07:54, Andrea Bolognani wrote:
> > Jim, can you please respin while reverting these changes that I had
> > suggested? Sorry :(
>
> No problem, will send a V7. BTW, what is the list preference for suppresscc?
> I've been meaning to ask that after the past several versions, as I watch
> git send-email spam the R-B. My global .gitconfig only suppresses 'self'.

Quoting [1]:

  As a rule, patches should be sent to the mailing list only: all
  developers are subscribed to libvir-list and read it regularly, so
  **please don't CC individual developers** unless they've explicitly
  asked you to.

(emphasis from the original).

IMHO we should remove that clause. There's no real harm in CC'ing
individual contributors if they're previously reviewed a vNN series
and it is a very normal thing for people todo in many email based
projects.  I don't want to see us complaining if contributors choose
to CC people.


IIRC the motivation for that clause was because some contributors cc'd
random people on v1.

That being said, I think the document is long enough even without the
clause so I sent a patch to drop it:
https://listman.redhat.com/archives/libvir-list/2023-January/236915.html

Jano

With regards,
Daniel
--
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux