Re: [PATCH] util: fix build without macvtap

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

 



  Michal Privoznik wrote:

> On 27.01.2016 03:45, Roman Bogorodskiy wrote:
> > Commit 370608b added new functions:
> > 
> >  - virNetDevMacVLanReleaseName,
> >  - virNetDevMacVLanReserveName.
> > 
> > Add stubbed versions of them to fix build without macvtap.
> > ---
> Oh, sorry. I started working on the same patch as soon as I got to the
> office and haven't checked for the e-mails just to find out that you
> already have a patch for that.
> 
> Next time you may as well just push the patch. There are basically two
> rules under which a patch can be pushed without review:
> 
> 1) Trivial - patch is just trivial (e.g. fixing an indentation, or a
> typo and stuff like that). Although, several trivial changes when
> combined create a non-trivial patch. So it's okay to push if commit is
> fixing few whitespaces, but fixing spelling from british to american in
> whole codebase will require proper review.
> 
> 2) Build breaker - if the fix is obvious and moreover it fixes a broken
> build, you may just push the patch. This situation is a good example of
> such case. Although the fix is trivial too.
> 
> Anyway, in both cases you must send the patch you've pushed to the list
> and note in comments that it's already pushed. The reason for that is so
> that upstream can check if the fix was good and if somebody has a better
> idea they can propose it.
> 
> Once again, sorry for taking over credit from you.

Ah, it's fine.

I'm aware of these rules and sometimes do push stuff right away, though
this time I was about to go to bed and wasn't planning to do anything
more, so it wasn't blocking me in any way, so I decided to just send it
over to make sure I didn't make any sleepy typo. ;-)

Roman Bogorodskiy

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[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]