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
signature.asc
Description: OpenPGP digital signature
