Print

Print

So, the question is whether to change the documentation or to change the code. The particular problem here is that the test should have been strlen(id) != 1 as opposed to '2'. The comment was a hold over after the following attempted fix:
11 November 2020
Correct documentation on how to specify the idtype (i.e. the single letter is not followed by a colon).

So, to keep things sane. The code will change to secretly allow two characters but only consider the first. That makes it backward compatible without needing the change the documentation as it was already changed but the code was not properly changed. Must have been a pandemic driven change, sigh.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <xrootd/xrootd/issues/2061/1681673342@github.com>

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/issues/2061#issuecomment-1681673342", "url": "https://github.com/xrootd/xrootd/issues/2061#issuecomment-1681673342", "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