Information needed for Qreator

[David Planella]

Al 19/05/12 21:10, En/na Ubuntu Software Center ha escrit:
> <https://myapps.developer.ubuntu.com/dev/>
> Your application Qreator Needs Information
> Thank you for your patience during the moderation process.
> We need some information
> 
> Please respond to this request from our approval team, and then resubmit
> your application:
> 
> So I ran 'quickly package --extras' in a branch of qreator. From what I
> see, there are a few things that aren't working correctly: 1) the
> desktop file is installed to
> /opt/extras.ubuntu.com/qreator/share/applications/ The system won't pick
> it up there. 2) The Exec and Icon lines in the desktop file don't point
> to the actually installed files in opt: Exec=qreator
> Icon=/usr/share/qreator/media/qreator.svg Even if you edit the icon path
> to be correct for opt, setup.py stomps on it since "datadir" isn't set
> correctly and this is called: for line in fin: if 'Icon=' in line: line
> = "Icon=%s\n" % (datadir + 'media/qreator.svg') fout.write(line) 3) The
> debian/changelog isn't given an appropriate version number for extras.
> 4) It sets --install-{lib, scripts, data} correctly, but it also needs
> to set --prefix since there's magic in setup.py like: self.prefix +
> '/share/qreator/' And some minor things picked up by lintian: P: qreator
> source: out-of-date-copyright-format-uri
> http://svn.debian.org/wsvn/dep/web/deps/dep5.mdwn?op=file&rev=135 Should
> be: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/ W:
> qreator source: debhelper-overrides-need-versioned-build-depends (>=
> 7.0.50~) W: qreator source:
> package-needs-versioned-debhelper-build-depends 8 W: qreator source:
> unused-build-dependency-on-cdbs W: qreator source:
> ancient-standards-version 3.8.3 (current is 3.9.3) Then, of course,
> lintian hates opt, but that's a known issue. I guess we should probably
> try to capture some of this as bug reports against quickly? I'm not sure
> what's already filed. I tried to note about everything, but might of
> missed something. I pushed a branch to:
> lp:~andrewsomething/qreator/arb-packaging It creates a package that
> build, installs, and works from opt on precise. It should be more or
> less ready. Oh, very cool app!
> 
> Visit the developer portal <https://myapps.developer.ubuntu.com/dev/>
> 
> 
>   Ubuntu developer <https://myapps.developer.ubuntu.com/dev/>
> 
> 
> 
> This body part will be downloaded on demand.

(replying here as there is no notification e-mail on feedback from
myapps. Just pasting my reply from [3])

Oh wow, thanks Andrew for the quick review and the fixes! Regarding the
issues with extras/opt packaging, there are some things that are known
[1] [2], but you've also discovered new issues. Would you mind reporting
them as separate bugs? I think it's important that the Quickly
developers are aware of them, as fixing them would make life for app
authors a lot easier. /opt still seems to be a major hurdle for getting
your apps into extras.

What's your suggestion going forward, shall I just merge your branch to
trunk? My only concern is that I'd like to keep using quickly commands
for packaging, as they come in quite handy for automating the uploads,
especially for those without much packaging skills such as myself, and
I'm worried that the quickly commands might overwrite the changes you've
made to make it work with /opt.

[1] https://lists.launchpad.net/quickly-talk/msg00080.html
[2] https://bugs.launchpad.net/quickly/+bug/998910
[3] https://myapps.developer.ubuntu.com/dev/apps/874/feedback/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 554 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/app-review-board/attachments/20120521/f9363516/attachment.pgp>