-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do not use private 'XrdCl::FileSystem::FileSystem(const XrdCl::FileSystem&) #11606
Conversation
…stem&)' The following showed up after a switch to xrootd 4.2.3 happened in CMSSW_7_6_DEVEL_X IBs. The ctor 'XrdCl::FileSystem::FileSystem(const XrdCl::FileSystem&)' is private and should not be used. Signed-off-by: David Abdurachmanov <[email protected]>
A new Pull Request was created by @davidlt for CMSSW_7_6_X. Do not use private 'XrdCl::FileSystem::FileSystem(const XrdCl::FileSystem&) It involves the following packages: Utilities/XrdAdaptor @cmsbuild, @smuzaffar, @Dr15Jones can you please review it and eventually sign? Thanks. |
Oh duh, good catch. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Could we get this in to revive DEVEL_X IBs? |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
Do not use private 'XrdCl::FileSystem::FileSystem(const XrdCl::FileSystem&)
The following showed up after a switch to xrootd 4.2.3 happened in
CMSSW_7_6_DEVEL_X IBs. The ctor 'XrdCl::FileSystem::FileSystem(const
XrdCl::FileSystem&)' is private and should not be used.
Signed-off-by: David Abdurachmanov [email protected]