@brianhlin : thanks for the PR! However, I do see few issues :
- the /run/xrootd directory is not packaged correctly, in the spec file we still have:
https://github.com/xrootd/xrootd/blob/ced832b45ed4d6a332dbc8006855ccda928255d6/packaging/rhel/xrootd.spec.in#L524
https://github.com/xrootd/xrootd/blob/ced832b45ed4d6a332dbc8006855ccda928255d6/packaging/rhel/xrootd.spec.in#L743
so need to replace `%{_var}/run` with `%{_rundir}`
- also, we need to still suport _rhel6_, so your changes need to be conditional and e.g. for slc6 we need to keep /var/run
Could you please update the PR?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/1032#issuecomment-530291087
########################################################################
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
