search

haskell-infra / hackage-trustees

Issue tracker for Hackage maintainance and trustee operations
https://hackage.haskell.org/packages/trustees/
42 stars 7 forks source link

HaXml Duration output is invalid #278

Closed dten closed 2 years ago

dten commented 3 years ago

https://hackage.haskell.org/package/HaXml-1.25.5/docs/src/Text.XML.HaXml.Schema.PrimitiveTypes.html#line-140

it's parsing the duration PT30M and outputting 0Y0M0DT0H30M0.0S which whilst being full of extra 0's is also invalid because it's missing the P which signifies it is a Period not a DateTime

phadej commented 3 years ago

Thanks for a report.

HaXml is not maintained, it is on a life-support. I'm hesitant to even review any functionality changing patches, and thus making Hackage Trustees de-facto maintainers of HaXml. If you are using HaXml and want to see it maintained, please take over its maintenance.

dten commented 3 years ago

Understandable.

The most interesting thing for me is generating the parser/serialiser from a xsl, which doesn't seem to be a thing that exists elsewhere.

By the way I think the fix is just add "P"++ before show y

It's definitely a bug also

I have thought about doing some PRs but am also reluctant to take on maintenance 😓

Bodigrim commented 2 years ago

@dten I very much suggest people to migrate away from HaXml, unless they are willing to take on maintenance. The package has been essentially abandoned 4 years ago, that's enough time to switch to something else. Hackage Trustees cannot support it forever. Doing NMUs for HaXml encourages people to use it further, which is quite an opposite direction to what I wish to happen.

Bodigrim commented 2 years ago

@dten please raise an issue at https://github.com/HaXml/HaXml