zericardo / sci-corpus

Scientific corpus manager.
6 stars 1 forks source link

We really need lxml module? #57

Closed dpizetta closed 10 years ago

dpizetta commented 10 years ago

If it works with xml module without changes, maybe it would be better we change to it because is one dependence less.

tiagocampo commented 10 years ago

The program doesnt work with only xml, it need lxml to properly export to XML format. Without lxml the resulting file will be hard to human read.

On Thu, May 22, 2014 at 5:05 PM, Daniel Pizetta notifications@github.comwrote:

If it works with xml module without changes, maybe it would be better we change to it because is one dependence less.

— Reply to this email directly or view it on GitHubhttps://github.com/zericardo182/sci-corpus/issues/57 .

                  Tiago de Campos
  ----------------------------------------------------------

                Skype: tiagocampo1
  https://www.facebook.com/tiagodecampo

dpizetta commented 10 years ago

Ok Im trying to avoid dependences. But ok.

On Thu, May 22, 2014 at 5:09 PM, tiagocampo notifications@github.comwrote:

The program doesnt work with only xml, it need lxml to properly export to XML format. Without lxml the resulting file will be hard to human read.

On Thu, May 22, 2014 at 5:05 PM, Daniel Pizetta notifications@github.comwrote:

If it works with xml module without changes, maybe it would be better we change to it because is one dependence less.

Reply to this email directly or view it on GitHub< https://github.com/zericardo182/sci-corpus/issues/57> .

Tiago de Campos

Skype: tiagocampo1 https://www.facebook.com/tiagodecampo

Reply to this email directly or view it on GitHubhttps://github.com/zericardo182/sci-corpus/issues/57#issuecomment-43937710 .