Hi Brian. I stay not a philosopher. The difference is that right now that
stuff reached production sites, a practical issue, so I wrote "I don't like it".
I also think that the solution proposed by Andreas solves the problem very well.
On 12/18/2018 01:12 PM, Brian P Bockelman wrote:
> @ffurano <https://github.com/ffurano> - I don't understand the opposition to breaking the ABI. In #585
> <https://github.com/xrootd/xrootd/issues/585>, I proposed making this API public in order to some sort of ABI stability for
> users (such as the DPM plugins) and you wrote:
>
> I am definitely not a philosopher of ABI compatibility, and I have put those headers with the private ones exactly to have
> more freedom, until the interface stabilizes, which can take quite some time.
>
> In that case, we broke the ABI for the 4.8.0 release to have more convenient access to the |XrdSecEntity|. So, everyone last
> year had to recompile their plugins to upgrade to the 4.8.0 release (just as we would have them doing here).
>
> If we can no longer do fixes to clear oversights the API, why don't we mark it as public?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub <https://github.com/xrootd/xrootd/pull/879#issuecomment-448200108>, or mute the
> thread <https://github.com/notifications/unsubscribe-auth/AFIaT6NsqIuW067ns7S3q2piIpMF-k5gks5u6NuwgaJpZM4ZR3iU>.
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/xrootd/xrootd","title":"xrootd/xrootd","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/xrootd/xrootd"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ffurano in #879: Hi Brian. I stay not a philosopher. The difference is that right now that\nstuff reached production sites, a practical issue, so I wrote \"I don't like it\".\nI also think that the solution proposed by Andreas solves the problem very well.\n\nOn 12/18/2018 01:12 PM, Brian P Bockelman wrote:\n\u003e @ffurano \u003chttps://github.com/ffurano\u003e - I don't understand the opposition to breaking the ABI. In #585\n\u003e \u003chttps://github.com/xrootd/xrootd/issues/585\u003e, I proposed making this API public in order to some sort of ABI stability for\n\u003e users (such as the DPM plugins) and you wrote:\n\u003e \n\u003e I am definitely not a philosopher of ABI compatibility, and I have put those headers with the private ones exactly to have\n\u003e more freedom, until the interface stabilizes, which can take quite some time.\n\u003e \n\u003e In that case, we broke the ABI for the 4.8.0 release to have more convenient access to the |XrdSecEntity|. So, everyone last\n\u003e year had to recompile their plugins to upgrade to the 4.8.0 release (just as we would have them doing here).\n\u003e \n\u003e If we can no longer do fixes to clear oversights the API, why don't we mark it as public?\n\u003e \n\u003e —\n\u003e You are receiving this because you were mentioned.\n\u003e Reply to this email directly, view it on GitHub \u003chttps://github.com/xrootd/xrootd/pull/879#issuecomment-448200108\u003e, or mute the\n\u003e thread \u003chttps://github.com/notifications/unsubscribe-auth/AFIaT6NsqIuW067ns7S3q2piIpMF-k5gks5u6NuwgaJpZM4ZR3iU\u003e.\n\u003e \n\n"}],"action":{"name":"View Pull Request","url":"https://github.com/xrootd/xrootd/pull/879#issuecomment-448211038"}}}
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/xrootd/xrootd/pull/879#issuecomment-448211038",
"url": "https://github.com/xrootd/xrootd/pull/879#issuecomment-448211038",
"name": "View Pull Request"
},
"description": "View this Pull Request 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
