BTW I just noticed that

https://github.com/apache/whimsy/blob/master/tools/collate_minutes.rb

does its own parsing of podlings.xml; maybe that should use podlings.rb instead?

On 26 February 2016 at 09:37, sebb <[email protected]> wrote:
> On 26 February 2016 at 00:31, Sam Ruby <[email protected]> wrote:
>> On Thu, Feb 25, 2016 at 6:26 PM, Sebastian Bazley <[email protected]> wrote:
>>> Commit a01897a8369e8c45fd4eedc2c04e813fe52e056c:
>>>     Add podling status if present
>>>
>>>
>>> Branch: refs/heads/master
>>> Author: Sebb <[email protected]>
>>> Committer: Sebb <[email protected]>
>>> Pusher: sebb <[email protected]>
>>>
>>> ------------------------------------------------------------
>>> www/roster/public_nonldap_groups.rb                          | +++++++++++
>>> ------------------------------------------------------------
>>> 11 changes: 11 additions, 0 deletions.
>>> ------------------------------------------------------------
>>>
>>>
>>> diff --git a/www/roster/public_nonldap_groups.rb 
>>> b/www/roster/public_nonldap_groups.rb
>>> index 2460147..f9dbd3f 100644
>>> --- a/www/roster/public_nonldap_groups.rb
>>> +++ b/www/roster/public_nonldap_groups.rb
>>> @@ -22,6 +22,15 @@
>>>  require_relative 'public_json_common'
>>>
>>>  require 'net/http'
>>> +require 'open-uri'
>>> +require 'nokogiri'
>>> +
>>> +PODLINGS = 'http://incubator.apache.org/podlings.xml'
>>> +pods = {}
>>> +podlings = Nokogiri::XML(open(PODLINGS))
>>> +podlings.search('podling').each do |podling|
>>> +  pods[podling['resource']] = podling['status']
>>> +end
>>
>> Could this be used instead:
>>
>> https://github.com/apache/whimsy/blob/master/lib/whimsy/asf/podlings.rb
>
> Yes, I assume so. Did not know it was there.
> Will look at it later.
>
>> And, if not, wouldn't it be better/faster/more reliable to use the
>> source in subversion than to fetch it over the web?
>
> It was a bit of hack to get the info out.
>
> Using SVN requires a bit more setup for local testing.
>
>> - Sam Ruby

Reply via email to