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.



Attachment: signature.asc
Description: OpenPGP digital signature

Pkg-mozext-maintainers mailing list

Reply via email to