[GitHub] ant-ivy issue #73: IVY-1104 Enable XML report parser to produce qualified ex...

2018-08-02 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/73 No dispute here; however, that may need some adjustment to XSL, which transforms XML report to HTML. --- - To unsubscribe,

[GitHub] ant-ivy issue #73: IVY-1104 Enable XML report parser to produce qualified ex...

2018-08-02 Thread aprelev
Github user aprelev commented on the issue: https://github.com/apache/ant-ivy/pull/73 To be honest, I do not like the idea of imposing implicit requirements on naming of namespaces. Escaping dashes in prefixes seems more robust and error-prone given correct codec is implemented. I

[GitHub] ant-ivy issue #73: IVY-1104 Enable XML report parser to produce qualified ex...

2018-08-02 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/73 Besides, `extra-` has to be kept for backwards compatibility. --- - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For

[GitHub] ant-ivy issue #73: IVY-1104 Enable XML report parser to produce qualified ex...

2018-08-02 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/73 `'-'` cannot be escaped since the spec does not permit

[GitHub] ant-ivy issue #73: IVY-1104 Enable XML report parser to produce qualified ex...

2018-08-02 Thread aprelev
Github user aprelev commented on the issue: https://github.com/apache/ant-ivy/pull/73 > Using a simple '-' as a separator may cause trouble if somebody decides to use a namespace prefix with '-'. I don't actually see a problem with that: we can - either escape dashes with

[GitHub] ant-ivy issue #73: IVY-1104 Enable XML report parser to produce qualified ex...

2018-08-02 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/73 Keeping `':'` would require XML report to deal with all those pesky namespaces, because it must be a valid XML. Using a simple `'-'` as a separator may cause trouble if somebody decides to

[GitHub] ant-ivy issue #73: IVY-1104 Enable XML report parser to produce qualified ex...

2018-08-01 Thread aprelev
Github user aprelev commented on the issue: https://github.com/apache/ant-ivy/pull/73 @twogee, why don't we just stick with deterministic encoding like ```Java extra- // e.g. extra-e:foo, extra-m:classifier, extra-foo ``` since both `'-'` and `':'` are allowed name