So, we has a discussion about this. Right now we are looking to see if we
can drop both requirements and use the builtin code, possibly renamed to
avoid rule conflicts. Right now I believe it will use the tinyxml version
as that's the default.
On Wed, 22 Feb 2023, Adrian Sevcenco wrote:
>> Actually it's more complicated than that. Originally we simply used our own version of tinyxml (see xrootd/src/XrdXml/tinyxml) and there were no explicit dependencies; though we allowed sites to optionally use libxml2 just in case there was a problem with our version of tinyxml. Then we ran into an EPEL rule that says that we cannot do that if a site has installed tinyxml and if it has then we are supposed to use the installed version. So, somewhere in that whole process these additional requirements were imposed but I don't know the history behind that. So, the answer is in reality there need not be any requirements but the distrib bureacracy says otherwise.
>
> so.. would that mean that tintxml is redundant and the only (prefered) requirement is libxml2?
> maybe this could use some clean up? i see in src/XrdXml/XrdXmlReader.cc:50 that if
libxml2 is present it will always be used but the cmake configuration allows both
to be present (and linked against?) which could be messy..
>
> So, would be possible to make them exclusive? (as a first step)
> Then, given that both are widely distributed, maybe just use only one of them?
> Thanks a lot!
>
> --
> Reply to this email directly or view it on GitHub:
> https://github.com/xrootd/xrootd/issues/1924#issuecomment-1439857608
> You are receiving this because you are subscribed to this thread.
>
> Message ID: ***@***.***>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.
Message ID: <xrootd/xrootd/issues/1924/1440783543@github.com>
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/xrootd/xrootd/issues/1924#issuecomment-1440783543",
"url": "https://github.com/xrootd/xrootd/issues/1924#issuecomment-1440783543",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1
