| Message ID | d1a1e265dd846de6ca40406300d91cecfba69ef8.1625769570.git.crobinso@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Move qemu.sasl to contrib/ | expand |
On Thu, Jul 08, 2021 at 02:39:57PM -0400, Cole Robinson wrote: > It's not installed in any way, so seems like contrib/ material Not really. Contrib is stuff that is included on an ad-hoc basis with no commitment to support and no guarantee that it is working at all. Yes, there is some stuff in contrib/ that doesn't belong there based on this definition, and should be moved out. The SASL config is most definitely supported, so doesn't belong in there. > > Signed-off-by: Cole Robinson <crobinso@redhat.com> > --- > Or should this be installed to /etc/sasl2 ? > > MAINTAINERS | 2 +- > qemu.sasl => contrib/qemu.sasl | 0 > 2 files changed, 1 insertion(+), 1 deletion(-) > rename qemu.sasl => contrib/qemu.sasl (100%) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 684142e12e..2ec1c362f0 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -2768,7 +2768,7 @@ F: tests/unit/test-crypto-* > F: tests/bench/benchmark-crypto-* > F: tests/unit/crypto-tls-* > F: tests/unit/pkix_asn1_tab.c > -F: qemu.sasl > +F: contrib/qemu.sasl > > Coroutines > M: Stefan Hajnoczi <stefanha@redhat.com> > diff --git a/qemu.sasl b/contrib/qemu.sasl > similarity index 100% > rename from qemu.sasl > rename to contrib/qemu.sasl > -- > 2.31.1 > 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 :|
On Thu, 8 Jul 2021 at 20:14, Daniel P. Berrangé <berrange@redhat.com> wrote: > > On Thu, Jul 08, 2021 at 02:39:57PM -0400, Cole Robinson wrote: > > It's not installed in any way, so seems like contrib/ material > > Not really. > > Contrib is stuff that is included on an ad-hoc basis with no > commitment to support and no guarantee that it is working at > all. Yes, there is some stuff in contrib/ that doesn't > belong there based on this definition, and should be moved > out. More generally, I would like to see us get rid of contrib/ entirely. We should either support stuff (and have it in sensible places in the source tree) or kick it out. thanks -- PMM
On 7/8/21 3:18 PM, Peter Maydell wrote: > On Thu, 8 Jul 2021 at 20:14, Daniel P. Berrangé <berrange@redhat.com> wrote: >> >> On Thu, Jul 08, 2021 at 02:39:57PM -0400, Cole Robinson wrote: >>> It's not installed in any way, so seems like contrib/ material >> >> Not really. >> >> Contrib is stuff that is included on an ad-hoc basis with no >> commitment to support and no guarantee that it is working at >> all. Yes, there is some stuff in contrib/ that doesn't >> belong there based on this definition, and should be moved >> out. > > More generally, I would like to see us get rid of contrib/ > entirely. We should either support stuff (and have it in > sensible places in the source tree) or kick it out. > qemu.sasl to ui/ dir then? Do you think we should install qemu.sasl to /etc/sasl2/qemu.conf? Its what Fedora/RHEL packaging does by hand for qemu. And what libvirt does for its equivalent config file Thanks, Cole
On Thu, Jul 08, 2021 at 04:49:24PM -0400, Cole Robinson wrote: > On 7/8/21 3:18 PM, Peter Maydell wrote: > > On Thu, 8 Jul 2021 at 20:14, Daniel P. Berrangé <berrange@redhat.com> wrote: > >> > >> On Thu, Jul 08, 2021 at 02:39:57PM -0400, Cole Robinson wrote: > >>> It's not installed in any way, so seems like contrib/ material > >> > >> Not really. > >> > >> Contrib is stuff that is included on an ad-hoc basis with no > >> commitment to support and no guarantee that it is working at > >> all. Yes, there is some stuff in contrib/ that doesn't > >> belong there based on this definition, and should be moved > >> out. > > > > More generally, I would like to see us get rid of contrib/ > > entirely. We should either support stuff (and have it in > > sensible places in the source tree) or kick it out. > > > > qemu.sasl to ui/ dir then? Yeah, probably best > Do you think we should install qemu.sasl to /etc/sasl2/qemu.conf? Its > what Fedora/RHEL packaging does by hand for qemu. And what libvirt does > for its equivalent config file Yes. 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 :|
diff --git a/MAINTAINERS b/MAINTAINERS index 684142e12e..2ec1c362f0 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2768,7 +2768,7 @@ F: tests/unit/test-crypto-* F: tests/bench/benchmark-crypto-* F: tests/unit/crypto-tls-* F: tests/unit/pkix_asn1_tab.c -F: qemu.sasl +F: contrib/qemu.sasl Coroutines M: Stefan Hajnoczi <stefanha@redhat.com> diff --git a/qemu.sasl b/contrib/qemu.sasl similarity index 100% rename from qemu.sasl rename to contrib/qemu.sasl
It's not installed in any way, so seems like contrib/ material Signed-off-by: Cole Robinson <crobinso@redhat.com> --- Or should this be installed to /etc/sasl2 ? MAINTAINERS | 2 +- qemu.sasl => contrib/qemu.sasl | 0 2 files changed, 1 insertion(+), 1 deletion(-) rename qemu.sasl => contrib/qemu.sasl (100%) -- 2.31.1