=== cpaelzer__ is now known as cpaelzer
[12:55] <ahasenack> is it acceptable for an ubuntu main package to have a recommends on an universe package?
[12:55] <ahasenack> in this case, depending on the configuration of the main package, that universe package might or might not be required
[12:55] <ahasenack> it's a plugin that can only be built and used with the universe package installed
[12:56] <ahasenack> the main package has many plugins, it's just one of many. All the others link to something in main, but this particular one, if enabled, would need universe
[12:56] <seb128> no, it's not
[12:56] <mdeslaur> it can be a suggests, but not a recommends
[12:56] <mdeslaur> (IIRC)
[12:56] <ahasenack> debian has it as a recommends
[12:56] <ahasenack> it's samba-vfs-modules, and the universe package is glusterfs-common
[12:57] <mdeslaur> you need to either move the plugin to a universe package, or stop building that particular plugin
[12:57] <ahasenack> https://salsa.debian.org/samba-team/samba/blob/master/debian/control#L304
[12:57] <ahasenack> right now we don't build it
[12:57] <ahasenack> but I keep getting bugs asking to enable it
[12:57] <ahasenack> there is a mir, let me check
[12:58] <ahasenack> it was nacked
[12:58] <ahasenack> https://launchpad.net/bugs/1274247
[12:58] <ubottu> Launchpad bug 1274247 in glusterfs (Ubuntu) "[MIR] Glusterfs" [High,Won't fix]
[13:00] <mdeslaur> and you can't make it a suggests instead of a recommends?
[13:00] <ahasenack> I suppose I could change our current delta, which is to not build it, to make it a suggests
[13:00] <mdeslaur> ie: if you build the module, and glusterfs-common isn't installed, will samba still work?
[13:00] <mdeslaur> ie: the module is disabled by default anyway, right?
[13:00] <ahasenack> mdeslaur: it will as long as you don't use the glusterfs module in smb.conf
[13:01] <ahasenack> yes, it's not loaded by default
[13:01] <mdeslaur> right, so make it a suggests instead of a recommends
[13:01] <ahasenack> so our delta would change from "drop gluster support" to "add gluster support as a suggests"
[13:01] <mdeslaur> right, or "move gluster recommends to suggests as it is in universe"
[13:02] <mdeslaur> whoever wants to use it just needs to install the missing package, but at least the module is built and availabe
[13:02] <mdeslaur> available
[13:03] <ahasenack> well, needs to know about it
[13:03] <ahasenack> there will be an error in the logs, smbd won't start, that kind of thing
[13:03] <mdeslaur> better than not having it at all ;)
[13:03] <ahasenack> I can check these conditions, but would an error be acceptable, if the module is used and gluster isn't installed?
[13:03] <mdeslaur> add something to the news file
[13:04] <mdeslaur> I think it's reasonable since the module isn't enabled by default
[13:04] <seb128> or split it to a new binary which is going to be in universe and have that depends on glusterfs-common?
[13:05] <mdeslaur> that's another possibility
[13:06] <ahasenack> yeah, something like samba-vfs-modules-glusterfs
[13:06] <ahasenack> would be a bigger delta from debian
[13:07] <ahasenack> thanks for the brainstorm
[13:08] <mdeslaur> np
[16:14] <rbasak> cjwatson: http://people.canonical.com/~cjwatson/ubuntu-policy/policy.html/ch-archive.html#s-universe doesn't have the build-depends against universe update.
[16:14] <rbasak> Would you like a bug filed? :-P
[16:17] <cjwatson> Sure
[16:17] <rbasak> Is there a LP project for it?
[16:26] <cjwatson> rbasak: https://launchpad.net/ubuntu/+source/ubuntu-policy
[16:26] <cjwatson> I'd be happy for somebody to take it over and merge it up
[16:27] <cjwatson> Should be gitified too
[16:30] <rbasak> Thanks!
=== jdstrand_ is now known as jdstrand
[17:47] <ahasenack> tjaalton: hi, the landscape-client sru uploads you rejected, can the new upload with the fix use the same version? I think it can, bceause the package never made it to proposed
[17:47] <ahasenack> https://bugs.launchpad.net/ubuntu/+source/landscape-client/+bug/1699179 for an example
[17:47] <ubottu> Launchpad bug 1699179 in landscape-client (Ubuntu Xenial) "PackageReporter kicks in during do-release-upgrade" [Medium,In progress]
[18:48] <ahasenack> Laney: that autopkgtest upgrade you mentioned in your email to ubuntu-devel@, does that apply to what bileto is using too?
=== ricab is now known as ricab|leaving
[20:33] <Laney> yes, there is but one place
=== Sir_Gallantmon is now known as Son_Goku
=== nOOb__ is now known as nOOb
[21:43] <tjaalton> ahasenack: yeah I think so