Looks like the perfect is the enemy of the good here.

I've put enough wasted time and effort into this. There is precious little
developer activity on Xerces-J.

I think we probably just have to officially fork Xerces-J if we want this
feature. Having the daffodil project maintain an official fork of Xerces-J
irks me as an ASF member.

Adding file/line/col information to the attribute nodes as mukulga suggests
is definitely a much more major change to Xerces-J's code base. I'm not
willing to consider doing it, as it isn't something we need.

But even if the Xerces-J team did merge my PR, who knows when there would
be a release with this feature in it.

I think we could build an XSD library in scala based on the existing scala
XML "constructing loader" in far less time than messing with XercesJ and
ws-schema.


---------- Forwarded message ---------
From: mukulga <notificati...@github.com>
Date: Wed, May 28, 2025 at 11:17 AM
Subject: Re: [apache/xerces-j] XERCESJ-1771 - location info feature (PR #7)
To: apache/xerces-j <xerce...@noreply.github.com>
Cc: Mike Beckerle <mbecke...@apache.org>, Mention <
ment...@noreply.github.com>


*mukulga* left a comment (apache/xerces-j#7)
<https://github.com/apache/xerces-j/pull/7#issuecomment-2916731309>

@mukulga <https://github.com/mukulga> This PR is still waiting for your
attention.

I'm sorry that due to other priorities these days, I'm not in a position to
merge this PR to Xerces-J dev repos branch any time sooner.
I also personally would like to have XML document location information,
present for other kinds of DOM nodes like attribute, comment etc. These are
also important XML document node types, and knowing their location via DOM
API would be useful. For the XML element's attributes, a workaround to get
attribute location as it's containing element's location doesn't look
appealing to me.
If another Xerces-J developer is fine with the implementation changes
produced by you for this PR as of now, they can merge your PR to relevant
Xerces-J branch and it would be fine to me.

—
Reply to this email directly, view it on GitHub
<https://github.com/apache/xerces-j/pull/7#issuecomment-2916731309>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALUDA57PB5HYLG26XFONLL3AXHR7AVCNFSM6AAAAABXPIKGF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDSMJWG4ZTCMZQHE>
.
You are receiving this because you were mentioned.Message ID:
<apache/xerces-j/pull/7/c2916731...@github.com>

Reply via email to