/srv/irclogs.ubuntu.com/2019/11/06/#ubuntustudio-devel.txt

EickmeyerOk, everyone, here's where we're at: I'm working on updating hydrogen to 1.0.0-beta1, which was "released" in March 2018. The reason being is that we need to remove Qt4, and this is the last bit.19:17
EickmeyerUnfortunately, I'm running into build errors, notably the "install" section.19:17
Eickmeyerteward: Perhaps I can enlist you to help? https://code.launchpad.net/~ubuntustudio-dev/+git/hydrogen19:17
Eickmeyerteward: As you can see, I'm running into FTBFS. Looks like it's occurring in the dh_install phase: https://launchpad.net/~eeickmeyer/+archive/ubuntu/ppa/+packages19:18
tewardgot build logs?19:18
EickmeyerYep.19:19
EickmeyerRight there in the link for every arch.19:19
tewardyour failures on amd64 are in the build package ohase19:19
EickmeyerOk, yes.19:20
EickmeyerLooks like in the doc section.19:20
Eickmeyerteward: Looks like I need to fix something in debian/rules, but I'm not sure what.19:21
tewardand specifically that portion of the rules are failing19:21
tewardlet me look19:21
Eickmeyerok19:21
tewardso it *looks* like: *** No targets specified and no makefile found.  Stop.19:24
tewardat what point does data/doc get built I wonder19:24
tewardlet me spin this in a local sbuild which I can interact with after a failure19:24
EickmeyerOk.19:25
tewardyou use gbp for this don't you :P19:29
tewardEickmeyer: Okay, so...19:38
tewardyou might be missing things in this19:38
tewardmissiong libraries in the output19:38
tewardunless you WANT to not have TAR, LADSPA, OSC, OSS support in this19:38
OvenWerksOSS support is questionable. The rest are more important19:39
tewardwell yer missing libraries for them19:40
tewardhttps://paste.ubuntu.com/p/yCbQhPJd6H/ <-- relevant output19:41
tewarderror reproduced on amd6419:41
Eickmeyerteward: TBH, the only thing I've done to the rules file is change it to point to Qt5. Before that, the last upload to debian was in 2016, so this rules file is pretty ancient.19:41
tewardokay so in amd64 the data/doc directory is created but there is no make file in it so make errors out19:45
tewardin THEORY we can remove that from d/rules19:45
tewardbut that would only fix amd6419:46
tewardi have to run other schroots to test the other archs19:46
tewardand you need a lot more build-deps for the otherbits you want to keep19:46
tewardfor the others, the dh_systemd_enable stuff is obsolete19:46
tewardyou need to use dh_installsystemd instead19:47
tewardthat error is selfexplanatory19:47
teward> dh_systemd_enable: dh_systemd_enable is no longer used in compat >= 11, please use dh_installsystemd instead19:47
tewardso you need to look up the differences between the two and how to rewrite the rules to use dh_installsystemd instead19:47
EickmeyerYeah, I couldn't figure out where dh_systemd_enable is even being called.19:49
Eickmeyerteward: ^19:50
tewardyeah not sure here either19:54
tewardoop i think i found it19:55
tewardit's in cdbs19:55
tewardit's in /usr/share/cdbs/1/rules/debhelper.mk line 241.19:56
tewardand that file is included in line 23 of the rules19:56
tewardI unfortunately cannot help you much with the CDBS stuff :/19:57
tewardmight ask for an assist in #ubuntu-devel maybe?19:57
tewardunless tsimonq2 wants to dig into cdbs hell.19:57
tewardso it looks like you have a lot of different problems, some are possibly CDBS introduced some aren't.  I haven't ever used CDBS for anything20:01
EickmeyerYikes. Keep me posted, I'll be monitoring @ubuntu-devel20:02
Eickmeyer*#20:02
Eickmeyers/@/#20:03
tewardEickmeyer OK so I think the problem is that it uses CDBS and not DH20:13
tewardwhich is going to introduce these issues20:13
EickmeyerOk, so that means we need to rewrite the rules file.20:13
tewardso unless we can remove CDBS from the equation (which we can't do unless we know CDBS, which I don't) I'm not sure how to proceed20:13
tewardyou'll need someone more experienced than me to fix this20:13
tewardalso note20:13
tewardhttps://bugs.debian.org/cgi-bin/bugreport.cgi?bug=874907#3820:13
ubottuDebian bug 874907 in src:hydrogen "[hydrogen] Future Qt4 removal from Buster" [Serious,Open]20:13
tewardnotice the last message there.  Probably a good idea to check with Debian on this, and see if they have a solution to get rid of CDBS and use short-form DH as was suggested20:14
EickmeyerUnfortunately tsimonq2 wants this done ASAP, so I'm not sure what to do now.20:15
tewardwe have two options:20:15
teward(1) tsimonq2 can assist with the CDBS -> DH transition and push the issue in Debian20:15
teward(2) remove hydrogen from consideration20:15
teward(and drop it, which sounds like it might be a 'bad idea'?)20:15
tewardtsimonq2: wake up and read20:16
teward*kicks Simon around*20:16
EickmeyerI'd rather go with option 1. Dropping Hydrogen would be bad.20:16
OvenWerksKind of like saying we don't do audio any more?20:17
tewardI unfortunately am not familiar with the package enough to do a CDBS -> DH transition, nor do I know CDBS that well either.  Unfortunately.20:18
tewardpretty sure SImon doesn't know cdbs either unless he's hiding something20:18
studiobot<teward001> @tsimonq2 wake up, time to sin!20:21
studiobot<teward001> :P20:21
tewardEickmeyer: also, you need to `update-maintainers` if there's an Ubuntu specific variation on this, system wouldn't let me gbp buildpackage it so I had to make a local change to make it behave :P20:24
tewardat least, just for consideration.  It should still work in Ubuntu/Debian but not when it FTBFS20:25
Eickmeyerteward: Ok.20:25
Eickmeyerteward: Done.20:26
EickmeyerOf course, if we don't end up having an Ubuntu version of this, it won't matter.20:27
tewardtrued20:28
tewardtrue*20:28
tewardsolution: fallback to compat 10 for now21:27
tewardper infinity in #ubuntu-devel21:27
Eickmeyerteward: yep, pushed. Building now.21:39
tewardcool.21:41
tewardlet me know if it FTBFS again21:41
tewardfor all we know this fixes all the issues xD21:41
Eickmeyerteward: Looks like it's working, but arm has yet to build.22:13
EickmeyerJust gotta clean-up a couple of lintian items, then we'll get it uploaded.22:17
studiobot<teward001> Nice.  I take it this isnt part of the packageset you have upload privs to?22:30
Eickmeyer@teward001: I don't have a packageset. So, no.22:36
EickmeyerOnly one who has the packageset is Ross, but you're a core dev so you've got even more access.22:36
tewardi have full upload rights lol23:08
Eickmeyerteward: Exaaaaaactly.23:08
tewardlet me know if all builds succeed AND the package worls23:08
tewardworks*23:08
tewardQt can be a pain so we want to make *sure* it works before I do any uploads23:08
EickmeyerBuilds succeed, but haven't tried the package yet. Got a lot going on.23:08
tewardand I"ll do some stringent testing on copyright, etc.23:09
tewardthat's the order of things23:09
tewardFORTUNATELY you have time :)23:09
EickmeyerExactly.23:09

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!