[03:16] <OvenWerks> teward: wrt subprocess.run, It doesn't do what I want it to do when I would have in the past used a command line with & at the end (works with shell=True of course but we don't want that)
[03:17] <teward> if you're trying to fork something into the background that can cause a problem
[03:17] <teward> OvenWerks: the other option is to just straight string call it but that's... not the best option either.  You could ask sarnold's opinion
[03:17] <teward> right now i'm heading to bed i'm tired
[03:17] <OvenWerks> teward: we have some "convenience" buttons that start things like a mixer
[03:17] <teward> good night.
[03:17] <OvenWerks> tomorrow
[03:18] <OvenWerks> I'll keep looking
[03:18] <teward> OvenWerks: we can revert that bit, if yuo really want it included in the NEW version, you or Eickmeyer can ask vorlon in #ubuntu-release to reject the upload on those grounds.
[03:18] <teward> but that's your guys call.
[03:18]  * teward needs sleep.  <off>
[03:29] <OvenWerks> subprocess.Popen(["/bin/mycmd", "myarg"]).pid did the trick.
[13:56] <teward> eesh popen's a low level operation but yeah...
[13:56] <teward> that can be headachey
[13:57] <teward> just tread lightly :p
[13:57] <teward> OvenWerks: if you need me to reupload let me know
[14:03] <teward> I asked AAs to NACK the package that WAS uploaded because of this bug
[14:03] <teward> Eickmeyer: OvenWerks: ***Test the hell out of this as is - no additional changes*** before I consider it ready for sponsoring again.
[14:44] <teward> no additioanl changes except bugfixes*
[14:54] <OvenWerks> teward: "the package" so far as I know ubuntustudio-menu-add was ok as of yesterday about noon. After that I started working on ubuntustudio-controls to remove all shell=True
 @ovenWerks are you saying that the subprocess.Popen call was not needed?
 it sounded from your messages that the subprocess.run was not behaving right
 and for some cases it probably won't
 but i need to know specifically (because your statements were unclear) as to the state of that package.
[14:55] <OvenWerks> teward: so all comments after about 12:30 (probably 1530 your time) are in reguard to -controls
 ack
 OK
 i wasn't sure ;)
 but i'll RESPONSOR this and ask seb to review it, then apologize to the AAs for the confusion
[14:56] <OvenWerks> There have been no changes to menu-add
[14:56] <teward> Eickmeyer: there was another two packages on the radar, weren't there?
[14:56] <teward> that you bugged me about
[14:59] <Eickmeyer> teward: Yes, dpf-plugins and pulseeffects (though that one isn't mine, just wanted to get the ball rolling for the dev).
[14:59] <teward> dpf-plugins *is* yours though yes?
[15:00] <Eickmeyer> teward: Well, I didn't write it, but I did the packaging.
 well we can say the same for ubuntustudio-menu-add too ;)
 but i mean it had your bootprints in it :P
[15:04] <teward> Eickmeyer: link me that dpf-plugins bug again here please?
[15:04] <teward> and sorry for switching between TG and IRC like i'm bouncing between things :p
[15:08] <Eickmeyer> teward: Standby...
[15:11] <Eickmeyer> teward: bug 1829562
[19:16] <teward> sarnold: you around?
[19:16] <teward> (I know the answer lol)
[19:16] <sarnold> teward: pong
[19:16] <teward> sarnold: teach Eickmeyer how to have the same version of a package in multiple releases per Security Patch Prep guidelines for version strings
[19:16] <teward> i.e.
[19:16] <teward> 3.0 in Eoan
[19:16] <teward> can't be 3.0 in Disco
[19:17] <teward> iut needs to be 3.0~0ubuntu0.18.10.1 etc.
[19:17] <sarnold> here's the table the security team uses https://wiki.ubuntu.com/SecurityTeam/UpdatePreparation#Packaging
[19:17] <sarnold> probably not-security-people have different rules but this should be pretty good to give the flavour of how it works :)
[19:17] <teward> sarnold: and for native packages without debrevs?
[19:17] <teward> in this case its' ubuntustudio-installer that is a bare version string
[19:18] <teward> and how that'd affect Eoan too
[19:18] <teward> AIUI you'd have to 'decrement' it in lower releases
[19:18] <teward> but i usually avoid native packages :)
[19:18] <sarnold> disco: 0.02, Pocket: release, Component: universe
[19:18] <sarnold> eoan: 0.04, Pocket: release, Component: universe
[19:18] <sarnold> these?
[19:18] <teward> sarnold: yep
[19:18] <teward> SRU request to 0.04 -> disco
[19:18] <teward> but sec team kinda sets guidelines on the v strings :)
[19:19] <teward> AH DAMN
[19:19] <teward> just spilled my coffee over me >.<
[19:20] <sarnold> uhoh
[19:20] <Eickmeyer> Dang.
[19:20] <teward> sarnold: how would the bare versions be handled for 'backports' direct
[19:20] <teward> not sure how SRU would handle that but from a version string side I see it being a problem
[19:21] <sarnold> ~18.04.1, ~18.10.1, etc probably
[19:21] <teward> yeah that's what i was going to say
[19:21] <teward> because 0.04~19.04.1 < 0.04
[19:21] <Eickmeyer> sarnold: Should I make it 0.04~19.04.1?
[19:21] <Eickmeyer> Oh, nm.
[19:21] <teward> for a disco sru you probably would need to yes.
[19:21] <teward> ASSUMING 0.04 is already in Eoan
[19:21] <Eickmeyer> Well, it's a native package, so that's an issue.
[19:22] <Eickmeyer> 0.04 is already in eoan. Uploaded it myself.
[19:22] <sarnold> and I don't know if the sru team would want an 'ubuntu' in there, or if this package exists only in ubuntu, if it's alright to skip it
[19:22] <teward> yeah this is one of those edge cases that I don't see coming up much
[19:22] <Eickmeyer> Package exists only in Ubuntu.
[19:22] <Eickmeyer> Completely native, not intended for Debian under any circumstance.
[19:24] <Eickmeyer> sarnold: If I put 0.04~19.04.1 since it's native, would that work?
[19:24] <teward> it should, native or not it's merely how dpkg pays attention to what version is where and such I think
[19:24] <teward> but that's probably something SRU might balk at
[19:25] <teward> not sure
[19:25] <Eickmeyer> Not the first time we've SRU'd something native. 
[19:26] <sarnold> I don't have a whole lot of experience with sru rules, but that version number would at least keep dpkg happy :)
[19:26] <teward> yep
[19:26] <teward> and wouldn't be rejected by the upload system
[19:28] <Eickmeyer> sarnold, teward: For example, the version of ubuntustudio-controls initially included in disco was 1.7. The SRU'd version is 1.7.1. So, what would happen if I made the SRU for ubuntustudio-installer 0.03? That hasn't been used yet.
[19:28] <sarnold> did launchpad ever see a version 0.03?
[19:28] <sarnold> launchpad won't let you reuse version numbers
[19:29] <Eickmeyer> sarnold: Oh, yes, you're right, it probably did see 0.03. So, if I used the ~19.04.1 suffix, that would do the trick?
[19:29] <teward> yes, it would
[19:29] <sarnold> yes
[19:29] <Eickmeyer> ok. I'll do that.
[19:30] <teward> Eickmeyer: 0.04 > 0.04~19.04.1 > 0.04~18.10.1 > ...
[19:30] <Eickmeyer> I see.
[19:30] <teward> which is a dpkg-ism with how ~ behaves in the string :)
[19:30]  * Eickmeyer will work on this when he gets back from some minor grocery shopping
[20:58] <Eickmeyer> OvenWerks: <seb128> teward, Eickmeyer, looking to ubuntustudio-menu-add there is no COPYING file nor mention of copyright holder or license outside of the debian dir, that seems suboptimal
[21:32] <OvenWerks> copyright ubuntustudio 2019, gpl2+
[21:33] <OvenWerks> (According to the about box)
[21:33] <Eickmeyer> OvenWerks: seb128 is referring to a /COPYING file and a /LICENSE file.
[21:34] <Eickmeyer> Though, this has never been a requirement for any of our other native packages, so I'm willing to appeal this one.
[21:34] <OvenWerks> Yeah, I have no idea how to create these.
[21:34] <OvenWerks> what format they should be etc.
[21:35] <Eickmeyer> I'm not 100% sure on that one. I think the /LICENSE file just needs to contain the entire text of GPL2.
[21:35] <OvenWerks> -controls has a COPYING file
[21:36] <OvenWerks> in -controls the COPYING file has that
[21:36] <Eickmeyer> Ok, then just copy the COPYING file and that should do it.
[21:40] <Eickmeyer> OvenWerks: Want me to do that?
[21:47] <Eickmeyer> OvenWerks: Nm, put all of that in. I'll let teward know it's done.
[21:49] <Eickmeyer> Done. teward: new upload ready.