[Pkg-mozext-maintainers] Bug#762310: Bug#762310: Bug#762310: Bug#762310: amo-changelog currently pointless

[Ximin Luo]

On 17/12/14 10:14, Ximin Luo wrote:
> On 01/10/14 02:05, Ximin Luo wrote:
>> On 21/09/14 02:12, David Prévot wrote:
>>> As can be seen in /usr/share/doc/xul-ext-password-editor/changelog.gz,
>>> the output is terrible: lists are not honored, and limited to the first
>>> page available on the website.
>>>
>>
>> Hello, please find attached 2 patches.
>>
>> 0001-amo-changelog-parse-into-human-readable-form-and-sav.patch
>>
>> This one changes amo-changelog to generate two files, debian/upstream/changelog{.html,} with the HTML directly from AMO and the plain version generated by lynx or pandoc, depending what the maintainer thinks looks best.
>>
>> I have tested this with status-4-evar, tree-style-tab, flashblock, noscript, saved-password-editor, adblock-plus, https-everywhere, firebug, public-fox and believe it produces pretty reasonable output for all of them. If you disagree, please make some constructive suggestions to move forward, instead of merely "it looks rubbish".
>>
>> 0002-amo-changelog-fetch-all-pages-of-Version-History.patch
>>
>> This one enables amo-changelog to automatically follow and download <link rel="next"> RSS references to the next page. This is dependant on AMO enabling support for this on their website, which is currently pending a pull request from me.
>>
>> https://github.com/mozilla/olympia/pull/301
>>
>> Please review the above patches, bearing in mind the second one is not directly testable yet.
>>
> 
> Ping? It seems AMO has deployed my changes to their website, so now multiple pages of the RSS feed works. I've just tested it out on tree-style-tab and it works nicely.
> 

OK, apparently I do have push rights to that repo, so I've gone ahead and pushed these patches, tweaking one of them to correct a path mistake.

In your own time, please review it, release a new version of mozilla-devscripts, and close this bug.

X

-- 
GPG: 4096R/1318EFAC5FBBDBCE
git://github.com/infinity0/pubkeys.git

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-mozext-maintainers/attachments/20141217/d0c05a2e/attachment.sig>