Remove brain damaged installation procedure for pari on OS X

[fbissey]

The current spkg-install script for pari includes the following specific darwing instruction

if [ "$UNAME" = "Darwin" ]; then
        # The following is a hack to get the library installed correctly
        # on Mac OS X:
        echo "Applying Mac OS X hack -- deleting and reinstalling libpari.a ..."
        rm -f Odarwin-ppc/*lib*
        $MAKE install-lib-sta
        if [ $? -ne 0 ]; then
            echo >&2 "Error reinstalling PARI's static libraries (OS X trick)"
            exit 1
        fi

        cd "`config/objdir`" && cp -f libpari* "$SAGE_LOCAL/lib"
        if [ $? -ne 0 ]; then
            echo >&2 "Error copying PARI's libraries (OS X trick)"
            exit 1
        fi
        cd "$CUR"
    fi

This section completely destroy the proper setting of darwin's install_name for no good reason. It needs to be removed. See conversation in #16017 and #16044 about this.

[fbissey]

Ready for review if you could Karl-Dieter...

---

New commits:

​53e6d2e

Remove counter productive installation procedure for darwin (OS X) it currently destroys the proper setting of OS X install_name.

[git]

Branch pushed to git repo; I updated commit sha1. New commits:

​62039fb

Forgot to bump the revision level

[kcrisman]

Ready for review if you could Karl-Dieter...

Haha! I might get a chance to do it on a couple older machines (10.4 and 10.7) eventually, but you should have reviewers testing it on 10.9 and 10.10, methinks.

[fbissey]

Replying to kcrisman:

Ready for review if you could Karl-Dieter...

Haha! I might get a chance to do it on a couple older machines (10.4 and 10.7) eventually, but you should have reviewers testing it on 10.9 and 10.10, methinks.

I got 10.10 covered before committing.

[kcrisman]

This passes all long doctests on 10.7 (other than the PowmInsecureWarning: Not using mpz_powm_sec. and a Maxima timeout, but those are unrelated and happen to me anyway).

Maybe someone should test on 10.9, but otherwise I wouldn't hold it up for me testing on any older platform.

[kcrisman]

John or William, can you quick check this doesn't break anything? I don't have access to the right version of OS X and François is of course the author here. Should be a very quick finish of review.

[fbissey]

pari has been updated in rc0 I need to rebase this.

[fbissey]

New branch pushed.

---

New commits:

​0fa866c

Trac #17588: Remove darwin specific install instruction that are counterproductive.

[fbissey]

No new code just made sure it would merge ok.

[kcrisman]

Just need someone to check this independently of you on 10.9 or 10.10 (preferred), then.

[jhpalmieri]

It certainly builds on OS X 10.10 with sage -f pari. I need to do make distclean; make, though, to ensure that it's really working. This is in progress. (If the installation procedure is being changed, then just doing sage -f pari might leave parts of the old installation lying around, so it feels safer to delete everything and start over.)

[fbissey]

It shouldn't as the file set installed is the same. Just the content of the dylib headers are different. But it is better to rebuild all pari dependencies to make sure it has been taken into account and rebuilding will do that for you.

[jhpalmieri]

This works for me on OS X 10.10.

[kcrisman]

All is well, then?