/srv/irclogs.ubuntu.com/2012/05/25/#ubuntu-app-devel.txt

twobottuxauappdev: Is it possible to add to Ubuntu Software Center *.jar? <http://askubuntu.com/questions/141939/is-it-possible-to-add-to-ubuntu-software-center-jar>04:21
twobottuxauappdev: How do I submit .jar file to the ubuntu software center <http://askubuntu.com/questions/141939/how-do-i-submit-jar-file-to-the-ubuntu-software-center>05:40
dholbachgood morning06:49
twobottuxauappdev: How to use gettext to make QML files translatable <http://askubuntu.com/questions/142012/how-to-use-gettext-to-make-qml-files-translatable>09:45
=== allison is now known as wendar
wendarmterry: sorry, wasn't on earlier. In general, if it can go in /opt, it should.17:17
=== wendar is now known as allison
=== allison is now known as wendar
mterrywendar, yar, I figured a way for it to go in opt.  Thanks!17:18
wendarmterry: cool. If you get a second, add a brief note on how you got it to work for /opt to the wiki documentation: https://wiki.ubuntu.com/AppReviewBoard/Submissions/InstallPathTips17:19
mterrywendar, the "Tips for python apps" link there is dead17:22
wendarmterry: I noticed that, checking if the link is just wrong17:22
wendarmterry: it looks like I never added the file, just added it now17:42
mterrywendar, I've got a big branch getting ready for SRU to fix extras support in quickly.  I was dismayed to see how broken it was!  I've added a bunch of tests to help make sure we don't do that again17:43
mterrywendar, we're generating metadata like "XB-Screenshot-Url: https://software-center.ubuntu.com/screenshots/t/testapp-quantal.png17:58
mterryXB-Thumbnail-Url: https://software-center.ubuntu.com/screenshots/t/testapp-quantal.thumb.png"  Is that still correct?17:58
wendarmterry: that can all disappear now17:58
mterrywendar, oh really?  XB-AppName, XB-Icon, all that?17:59
wendarmterry: they've made some changes in MyApps, so Software Center can now pull that data from the MyApps submission17:59
wendarmterry: no need to add it to the package files17:59
mterrywendar, OK.  You also recommend things like Standards 3.9.2, debian/source/format, and dropping cdbs.   How much do you really care about that?  (like is format 1.0 a blocker, is cdbs depends but not being used a blocker)18:00
wendarmterry: the debian/rules file has been simplified too, so it no longer needs ('icons', glob.glob('icons/*.png')),18:00
wendarmterry: er, that's no longer needs override_dh_gencontrol18:01
mterrywendar, you mean the "dpkg-distaddfile testapp.svg raw-meta-data -" bit?18:01
wendarmterry: we'll rip out cdbs if it comes through, and will update the standards version to the recommended version for the distro release18:02
wendarmterry: (IIRC, that is 3.9.2 for Oneiric)18:02
wendarmterry: yes, the raw-meta-data bit18:02
wendarmterry: not needed anymore18:02
mterrywendar, ok, can drop that too18:02
wendarmterry: if you're using the simple dh template recommended on the ARB wiki pages, it doesn't use cdbs anyway18:03
wendarmterry: so the dependency is unnecessary18:03
mterrywendar, agreed, it's just that those nits I described come from a separate package.  So if we really wanted to get rid of them, I'd have to add code to quickly to change them18:05
wendarmterry: which nits? We're fine on manual package generation now, either with dh_make, or dumping simple template files.18:06
wendarmterry: doesn't seem like we need to edit any other packages to get it working.18:06
wendarmterry: just change how quickly spits out the package templates18:07
mterrywendar, quickly uses python-mkdebian from python-distutils-extra to generate its debian/* files18:07
wendarmterry: long-term, we can get around that by using pkgme instead18:08
mterrywendar, agreed.  But my concern right now is 12.0418:08
wendarmterry: but, for the moment, can you just skip python-mkdebian for ARB packages18:08
wendarmterry: use static templates18:08
wendarmterry: the ARB python packages are my definition *very* simple18:09
wendar"by definition"18:09
wendarthey don't need to detect a lot of complex options18:09
mterrywendar, um...  they are if the developer has changed their actual code to support only ARB style.  To support normal style as well as ARB, let me show you the current debian/rules I'm using in my extras branch18:10
mterrywendar, http://pastebin.ubuntu.com/1006807/18:10
mterryNot that simple18:10
wendarmterry: taking a look18:13
wendarmterry: ugh, yeah, override_dh_install is pretty terrible18:14
wendarmterry: why is it so complex? --prefix should take care of it all18:14
wendarmterry: that's python's way of changing the path on everything18:14
wendarmterry: maybe the Quickly setup.py script needs to be modified to respect --prefix?18:15
mterrywendar, we have to edit the desktop file, we have to compile the schemas manually because the normal trigger doesn't fire, we have to point to a special XDG_DATA_DIRS so GNOME picks up the write schemas and help files, etc18:15
mterryWe want to fix the gettext usage to point to our translations...18:16
wendarmterry: that could all go in setup.py18:16
mterrySure, if you have an ARB-only app.  But that's not how quickly is constructed currently.  Certainly not for 12.04.18:17
mterrywendar, the decision of whether to build for ARB is made at debian package time18:17
wendarmterry: right, by passing the --prefix option to setup.py18:17
mterrywendar, again, none of the things I mentioned above are fixed by --prefix18:18
wendarmterry: but they *could* be, if setup.py handled the --prefix option appropriately18:18
mterry(unless I'm horribly wrong about that -- I don't believe that changes where gettext points and such)18:18
wendarmterry: the changes aren't specific to ARB, they'd have to be made for any installation on a different path18:19
wendarmterry: if, for example, a user wanted to install the package in /home/me/my/custom/path...18:19
mterrywendar, ah, I certainly agree.  But I think you are discussing this with a view to the future.  I'm discussing this with a view to fixing 12.04 to make quickly rock for ARB18:20
wendarmterry: I added some more details to https://wiki.ubuntu.com/AppReviewBoard/Submissions/PythonTips, with links to the python documentation18:20
wendarmterry: well, we don't want to review all that stuff in override_dh_install18:21
wendarmterry: if it was anything other than a Quickly package, we'd probably reject it if they needed that many install hacks in debian/rules18:21
mterrywendar, guh, hmm18:21
wendarmterry: but, if we need a "pass" for Quickly in 12.04, that's okay18:21
wendarmterry: we'll aim to make it rock even more in 12.10 :)18:22
wendarmterry: my advice, aim for a setup.py that's no more complex than the example on the PythonTips wiki page18:22
wendarmterry: arbitrary code in setup.py is a big "Red Flag" for python modules18:23
wendarmterry: (that's for 12.10)18:23
wendarmterry: oh... you know, we made some changes to some older Quickly submissions and didn't need all that... let me look for the diffs...18:26
mterrywendar, I think that simple of a setup.py is unrealistic.  You'll probably need to at least also edit fields in the desktop file and compile the schema.  But I agree that the simpler the better18:26
mterrywendar, but for 12.04.  So far the code I have generates that awful debian/rules file and also does not hit the following recommendations you have on your QuicklyTips page:  cdbs depend, standards version, debian/source/format, and older format for debian/copyright.  Without those fixes, is this all even saving you any effort?18:29
wendarmterry: it looks like --prefix does work18:30
wendarhttp://bazaar.launchpad.net/~allison/+junk/opensinhalease-precise/revision/218:30
mterrywendar, sorry, what do you mean by work?  I'm not arguing that --prefix isn't helpful, just that it doesn't do all the things you need it to, so you'll need special setup.py logic regardless18:31
wendarmterry: as in, setting --prefix gets Quickly to install everything under /opt18:32
mterrywendar, sure, agreed.  But things won't actually work when you run them18:32
wendarmterry: and then it only needs specific code to convince it to move the .desktop file out of /opt18:32
wendarmterry: yes, there were a couple places in the code where it needed to check if the filepath was /opt18:33
wendarmterry: but, it could do that and work for both /opt installation and normal installation18:33
mterrywendar, in addition to code changes to support arbitrary prefixes (like gettext, xdg_data_dirs, media file lookups), you will need to edit the desktop file and compile the schemas.  Those are not code changes, so will have to live in setup.py18:34
wendarsetup.py is already editing the .desktop file, setting it by --prefix18:34
mterrywendar, in a quickly package?  Yes (though it doesn't adjust Exec in 12.04, which is a problem)  But not in your ideal PythonTips page, which is what I was warning about.  Just that such a simple setup.py is unrealistic.  You'll need desktop editing code and schema compilation code18:36
wendarmterry: ah, I get you now18:36
wendarmterry: yes, there is some additional work required in setup.py18:37
wendar(for quickly)18:37
wendarmterry: I don't want to make this stressfull :) Make whatever hacks you need to make in debian/rules for 12.04, and aim to push them into setup.py for 12.1018:37
mterrywendar, well, for any app not only targetted at ARB.  Obviously if they only target ARB, they can take all sorts of shortcuts18:37
wendarmterry: and, yes, *any* changes you can make to Quickly so the generated packages require less manual editing is a big help18:38
wendarmterry: if we can get it to "no manual edits required" that would be awesome, but even just getting closer would help18:38
wendarmterry: the example you sent, with the lengthy override_dh_install, should be fine18:39
wendarmterry: it looks like it'll get the packages to the point where they aren't breaking any ARB policy18:39
mterrywendar, except for packaging nits, yeah18:40
mterrywendar, OK, will settle where I am for 12.04 then, get it reviewed and landed on trunk, then SRU'd18:40
wendarmterry: yup, that's good18:40
mterrywendar, thanks for your time!  Hopefully quickly will actually be useful for ya'll again18:41
wendarmterry: thanks much for working on quickly!18:41

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