-
Notifications
You must be signed in to change notification settings - Fork 149
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
XrdPosix.hh not installed #479
Comments
Hmm, that is strange. I will have to research the history of that commit
to see why the header file was removed from the public headers.
Andy
…On Fri, 10 Mar 2017, suhlatwork wrote:
Hi,
the file `XrdPosix.hh` is not copied to `include/XrdPosix` when XRootD is installed. Digging through the git history, it seems that this is the case since commit 67ab0ed. Given that this commit is already several years old with no one complaining about this so far, I wonder whether this was done on purpose or is a simple oversight.
This missing header means one cannot simply use the POSIX file functions like `open`, ..., but has to write `XrdPosix_Open`, ... in code making use of XrdPosix.
Thanks for the clarification!
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
#479
########################################################################
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
|
Why would you need this header? You can use the interface via preload or direct linking... |
Because some people did not want to do either and the header file gves you
that capability. It used to be public aand we couldn't find a reason why
it was removed from public and couldn't think of a reason why it shouldn't
be public.
…On Wed, 22 Mar 2017, Lukasz Janyst wrote:
Why would you need this header? You can use the interface via preload or direct linking...
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
#479 (comment)
########################################################################
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
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
the file
XrdPosix.hh
is not copied toinclude/XrdPosix
when XRootD is installed. Digging through the git history, it seems that this is the case since commit 67ab0ed. Given that this commit is already several years old with no one complaining about this so far, I wonder whether this was done on purpose or is a simple oversight.This missing header means one cannot simply use the POSIX file functions like
open
, ..., but has to writeXrdPosix_Open
, ... in code making use of XrdPosix.Thanks for the clarification!
The text was updated successfully, but these errors were encountered: