This repository has been archived by the owner. It is now read-only.

Convert remaining scripts to /bin/sh #38

Open
wants to merge 6 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@olebole
Contributor

olebole commented May 5, 2017

The mklibs, mkclean and getarch scripts in the different vendor subdirs are still /bin/csh, which would require the installation of the C-shell to build IRAF.

This patch converts them all to bin/sh.

I tested them locally, and all but vendor/mkpkg work well for me. The one person in the world who is able to run a complete build on an unchanged IRAF source tree is hereby asked to test if the build works as well within the IRAF build, and then to merge.

@RobSteele49

This comment has been minimized.

Show comment
Hide comment
@RobSteele49

RobSteele49 May 5, 2017

PR looks good to Steele. Of course my testing is for linux64 only.

RobSteele49 commented May 5, 2017

PR looks good to Steele. Of course my testing is for linux64 only.

olebole added some commits May 6, 2017

Fix toplevel location for curl build
libcurl.a shall be installed in vendor/voclient/lib, not in
vendor/voclient/common/lib. Otherwise it is not found later when
building the voapps.
@olebole

This comment has been minimized.

Show comment
Hide comment
@olebole

olebole May 7, 2017

Contributor

I just added a small fix that is needed to resolve #26: to build the VO apps, libcurl.a needs to be installed in vendor/voclient/lib and not in vendor/voclient/common/lib, which is done by adjusting the top variable in the mklibs script.

Contributor

olebole commented May 7, 2017

I just added a small fix that is needed to resolve #26: to build the VO apps, libcurl.a needs to be installed in vendor/voclient/lib and not in vendor/voclient/common/lib, which is done by adjusting the top variable in the mklibs script.

@olebole olebole changed the title from Convert mklibs to /bin/sh to Convert remaining scripts to /bin/sh Jan 23, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.