EUDAT-B2STAGE / B2STAGE-GridFTP

B2STAGE service core code for EUDAT project: iRODS-DSI
14 stars 15 forks source link

Minor bug fixes for iRODS connection setup #1

Closed vladimir-mencl-eresearch closed 9 years ago

vladimir-mencl-eresearch commented 9 years ago

Hi,

I have tried deploying this code and ran into several minor issues, primarily that the home directory was getting set wrong, to "/(null)/home//(nu".

I had a look and made a few small fixes - these are:

I hope you can accept these fixes as a contribution.

Please let me know if you'd like me to clarify any of the code changes.

Cheers, Vlad

Vladimir Mencl, Ph.D. E-Research Services and Systems Consultant BlueFern Computing Services University of Canterbury Christchurch New Zealand

http://www.bluefern.canterbury.ac.nz mailto:vladimir.mencl@canterbury.ac.nz

muccix commented 9 years ago

Hi Vladimir,

I have just accepted your pull request. Thanks again for your contribution.

Because of our home directory organization ( irods_zone/home/project_name/user_name), I have to free the user_name before setting the home_dir variable.

Later I can imagine to make that part configurable at compile time.

Best regards, Roberto

vladimir-mencl-eresearch commented 9 years ago

Hi Roberto,

I have moved this conversation into a separate issue: #2

Cheers, Vlad