Hotfix for MAD-X installation on Travis

pymad / cpymad

cPyMAD is a python interface to Mad-X using cython and libmadx

http://cern.ch/pymad

Apache License 2.0

3 stars 3 forks source link

Hotfix for MAD-X installation on Travis #48

Closed coldfix closed 10 years ago

coldfix commented 10 years ago

For whatever reason, the headers are only part of the installation if the patch level differs from MADX_PATCH_LEVEL=00. I guess, with the new release number this makes problems(?). The fix is copied from the AUR package.

Note that this effects only the Travis build and doesn't need to be tested in the testing branch. If you agree, I merge this ASAP, since it's needed for other branches as well.

Eothred commented 10 years ago

Hi,

This was an old design choice by me so that a dev and pro version of Mad-X would install on the same prefix without any conflicts.. But I finally decided it is simply stupid :) Removed in Mad-X svn revision 4693, so now headers are always installed.

Eothred commented 10 years ago

... Hence, you can decide to merge this (I don't mind), or ignore it since it should not be needed anymore...

Eothred commented 10 years ago

Did not read the solution. This patch will now fail. So I close this request then.

coldfix commented 10 years ago

Great! Thanks.