Re: [libvirt PATCH] qemu: reject readonly attribute for virtiofs

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

 



On a Wednesday in 2020, Daniel P. Berrangé wrote:
On Wed, May 13, 2020 at 11:51:50AM +0200, Peter Krempa wrote:
On Wed, May 13, 2020 at 10:41:23 +0100, Daniel Berrange wrote:
> On Wed, May 13, 2020 at 11:34:05AM +0200, Andrea Bolognani wrote:
> > On Wed, 2020-05-13 at 10:58 +0200, Ján Tomko wrote:
> > > This is not yet supported by virtiofsd.
> > >
> > > Fixes #23
> >
> > Please include the full URL here:
> >
> >   https://gitlab.com/libvirt/libvirt/-/issues/23

While I definitely don't like gitlab's URIs ...

> Using the '#23' format is the recommended way. GitLab UI will
> turn it into a hyperlink, and automatically close the mentioned
> issue.

... just mentioning #23 is useless for users looking at the git repo
directly (not through the web-ui) or if we at some point decide to
abandon gitlab.

Even if gitlab can't handle the automation of closing the comment when
the URI is mentioned I don't see why we should make it harder for users
who want to stay away from the browser as much as possible.

The point is for libvirt to follow normal practice from GitLab, so that
contributors don't have to know about libvirt specific rules for contributing
to the project. Telling users to change the normal issue syntax into a URL
whenever they send a patch is not something we should be doing.


Okay, I'll roll a die and push it one of with:
1) no issue reference
2) #23
3) https://gitlab.com/libvirt/libvirt/-/issues/23
4) #23 and https://gitlab.com/libvirt/libvirt/-/issues/23
5) https://www.youtube.com/watch?v=dQw4w9WgXcQ
6) https://gitlab.com/libvirt/libvirt/-/issues/23 and #23

...

Jano

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