Bring back Nokogiri parser about activewarehouse-etl

[byrnejb]

I have a nokogiri parser working in my fork, of which you are aware.

[thbar]

Yup! Thanks - I wrote this as a note to be sure I won't remember. I'll definitely bring your parser (I'm going through the fork queue, 166 commits, currently :-).

If later on you have something interesting, please send a pull-request!

[byrnejb]

On Fri, June 10, 2011 13:06, thbar wrote:

Yup! Thanks - I wrote this as a note to be sure I won't remember.
I'll definitely bring your parser (I'm going through the fork queue,
166 commits, currently :-).

If later on you have something interesting, please send a
pull-request!

Reply to this email directly or view it on GitHub:
#6 (comment)

I will be returning to work with AW_ETL sometime later this year.
At that time I will have the motivation to dig into the code again.

For our purposes AW-ETL is a one-time conversion utility to migrate
historical data from a long-lived business app into the new system
currently under development. Once the conversion is done we foresee
no further use. Consequently, like any disposable item, if it works
well enough to accomplish that then it is good enough.

*** E-Mail is NOT a SECURE channel ***
James B. Byrne mailto:[email protected]
Harte & Lyne Limited http://www.harte-lyne.ca
9 Brockley Drive vox: [phone]
Hamilton, Ontario fax: [phone]
Canada L8E 3C3

[thbar]

I hope it will work well enough - in any case, please report issues if you have :)

[byrnejb]

On Fri, June 10, 2011 13:21, thbar wrote:

I hope it will work well enough - in any case, please report issues
if you have :)

Reply to this email directly or view it on GitHub:
#6 (comment)

If you are going to cherry pick commits from my repo you had better
take a look at this file as well:

https://github.com/byrnejb/activewarehouse-etl/blob/master/lib/etl/commands/etl.rb

I rewrote the options parser and I believe that some of the stuff I
implemented elsewhere in AW-ETL depends upon this.

*** E-Mail is NOT a SECURE channel ***
James B. Byrne mailto:[email protected]
Harte & Lyne Limited http://www.harte-lyne.ca
9 Brockley Drive vox: [phone]
Hamilton, Ontario fax: [phone]
Canada L8E 3C3

[thbar]

I'll have a look, thanks!

Btw, one test fails in the nokogiri commit (test_nokogiri_xml_reader_parser_for_all_nodes) - nokogiri_reader.ctl isn't in git. Do you have it on your machine maybe ?

[thbar]

Aaah the NokogiriXmlReaderParser class isn't there - I can just delete the test.