Print

Print


@ktf : OK, fair enough, but your PR is setting C++17 for everyone not only for your stack, please keep in mind that we still support some platforms where C++17 is not available, hence this is a no-go. But fear not, we will do our best to accommodate your use case, we just need to find the right way of doing it :-) I do agree with @matthewfeickert that we probably need to use CXX_STANDARD similarly as root does.

As a side note, we also have those that probably need updating:
https://github.com/xrootd/xrootd/blob/d54b632c2291ad563b596efe7365e0271c9ef7ee/bindings/python/setup.py.in#L11-L19


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/1660/1075138660@github.com>

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