Merge lp:~sevoir/firefox-extensions/ubuntu-hu-menu.ubuntu into lp:~ubuntu-dev/firefox-extensions/ubuntu-hu-menu.ubuntu

[Alexander Sack]

[Alexander Sack (asac)]

requesting merge against empty ~ubuntu-dev branch so we can get review discussion here.

[Alexander Sack (asac)]

so obvious comment is that you didnt use med-xpi-unpack to produce your .upstream branch. in particular there must not be a .jar file etc. i will check more later.

review: Needs Fixing

[lp:~sevoir/firefox-extensions/ubuntu-hu-menu.ubuntu]

11. By Szenográdi Norbert Péter on 2009-10-01

* unpacking jar file *

[Szenográdi Norbert Péter (sevoir)]

> so obvious comment is that you didnt use med-xpi-unpack to produce your
> .upstream branch. in particular there must not be a .jar file etc. i will
> check more later.

unpacked and uploaded.

[Benjamin Drung (bdrung)]

The same comments as for ubuntu-hu-menu:

* The source package should not contain xul-ext (simply call it ubuntu-hu-menu)
* The long description is a little bit short
* I recommend to use DEP-5 for the copyright file: http://dep.debian.net/deps/dep5/
* Remove the comments from the debian/rules file

Is MOZ_XPI_EMID really required? If it does not work without it, it is probably a bug in mozilla-devscripts.

review: Needs Fixing

[Alexander Sack (asac)]

Benjamin, maybe you can point sevoir to an example for dep5 copyright. i think reading through the whole spec is too much as most extensions probably have everything the same license, etc.

review: Abstain

[Benjamin Drung (bdrung)]

Here are some examples:
http://changelogs.ubuntu.com/changelogs/pool/universe/a/abraca/abraca_0.4-2/abraca.copyright
http://changelogs.ubuntu.com/changelogs/pool/universe/p/promoe/promoe_0.1.0-1/promoe.copyright
http://changelogs.ubuntu.com/changelogs/pool/universe/p/pwdhash/pwdhash_1.7-5ubuntu1/pwdhash.copyright

review: Abstain

Unmerged revisions

11. By Szenográdi Norbert Péter on 2009-10-01

* unpacking jar file *

10. By Szenográdi Norbert Péter on 2009-10-01

* jar file link correction *

9. By Szenográdi Norbert Péter on 2009-10-01

* debian/rules correction *

8. By Szenográdi Norbert Péter on 2009-10-01

* debian/rules correction *

7. By Szenográdi Norbert Péter on 2009-10-01

* jar file correction *

6. By Szenográdi Norbert Péter on 2009-10-01

* some editing *

5. By Szenográdi Norbert Péter on 2009-10-01

* tuttoolbar.jar correction *

4. By Szenográdi Norbert Péter on 2009-10-01

* editing debian/control *

3. By Szenográdi Norbert Péter on 2009-10-01

* editing debian/rules *

2. By Szenográdi Norbert Péter on 2009-10-01

* initial packaging based on XPI.TEMPLATE *