Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PATCH] configuring ranlib for perl on osx with xcode 2.1 #8001

Closed
p5pRT opened this issue Jul 1, 2005 · 9 comments
Closed

[PATCH] configuring ranlib for perl on osx with xcode 2.1 #8001

p5pRT opened this issue Jul 1, 2005 · 9 comments

Comments

@p5pRT
Copy link

p5pRT commented Jul 1, 2005

Migrated from rt.perl.org#36448 (status was 'resolved')

Searchable as RT36448$

@p5pRT
Copy link
Author

p5pRT commented Jul 1, 2005

From lukec@ActiveState.com

Hello,

It looks like Apple has changed how ar works in xcode 2.1. As a
result, Configure thinks that ar can generate random libraries.

This breaks linking libperl.a, for instance when building PerlApp.

Configure doesn't currently allow overriding ranlib when Configure
detects that ranlib isn't necessary. My patch fixes this, and sets
ranlib for darwin.

Note that this patch is for ActivePerl 5.6.1 (it doesn't appear to be
fixed in 5.9.2), so it probably needs to be manually applied...

==== Configure ====
13888c13888,13890
< ranlib="​:"


    if \[ "X$ranlib" = "X" \]; then
        ranlib="&#8203;:"
    fi

==== hints/darwin.sh ====
179a180,183

# Configure doesn't detect ranlib on Tiger properly.
# NeilW says this should be acceptable on all darwin versions.
ranlib='ranlib'

Luke

--
Luke Closs
PureMessage Developer
There is always time to juggle in the Sophos Zone.

@p5pRT
Copy link
Author

p5pRT commented Sep 11, 2005

From @smpeters

[lukec@​ActiveState.com - Fri Jul 01 14​:56​:29 2005]​:

Hello,

It looks like Apple has changed how ar works in xcode 2.1. As a
result, Configure thinks that ar can generate random libraries.

This breaks linking libperl.a, for instance when building PerlApp.

Configure doesn't currently allow overriding ranlib when Configure
detects that ranlib isn't necessary. My patch fixes this, and sets
ranlib for darwin.

Note that this patch is for ActivePerl 5.6.1 (it doesn't appear to be
fixed in 5.9.2), so it probably needs to be manually applied...

==== Configure ====
13888c13888,13890
< ranlib="​:"
---

    if \[ "X$ranlib" = "X" \]; then
        ranlib="&#8203;:"
    fi

==== hints/darwin.sh ====
179a180,183

# Configure doesn't detect ranlib on Tiger properly.
# NeilW says this should be acceptable on all darwin versions.
ranlib='ranlib'

Luke

Has anyone looked at this patch yet? I'm assuming that since it was
written against 5.6.1 rather than the current bleadperl and it isn't in
diff -u format, a new patch might be needed.

One question comes to mind when I look at it, though. Have you tested
the patch on a non-Tiger system?

@p5pRT
Copy link
Author

p5pRT commented Sep 11, 2005

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Sep 11, 2005

From @rgarcia

On 9/11/05, Steve Peters via RT <perlbug-followup@​perl.org> wrote​:

Has anyone looked at this patch yet? I'm assuming that since it was
written against 5.6.1 rather than the current bleadperl and it isn't in
diff -u format, a new patch might be needed.

One question comes to mind when I look at it, though. Have you tested
the patch on a non-Tiger system?

Yes. I wonder whether the same effect can be achieved only by patching
the Darwin hints file.

@p5pRT
Copy link
Author

p5pRT commented Sep 13, 2005

From lukec@ActiveState.com

On Sat, Sep 10, 2005 at 10​:40​:25PM -0700, Steve Peters via RT wrote​:

[lukec@​ActiveState.com - Fri Jul 01 14​:56​:29 2005]​:

Hello,

It looks like Apple has changed how ar works in xcode 2.1. As a
result, Configure thinks that ar can generate random libraries.

This breaks linking libperl.a, for instance when building PerlApp.

Configure doesn't currently allow overriding ranlib when Configure
detects that ranlib isn't necessary. My patch fixes this, and sets
ranlib for darwin.

Note that this patch is for ActivePerl 5.6.1 (it doesn't appear to be
fixed in 5.9.2), so it probably needs to be manually applied...

==== Configure ====
13888c13888,13890
< ranlib="​:"
---

    if \[ "X$ranlib" = "X" \]; then
        ranlib="&#8203;:"
    fi

==== hints/darwin.sh ====
179a180,183

# Configure doesn't detect ranlib on Tiger properly.
# NeilW says this should be acceptable on all darwin versions.
ranlib='ranlib'

Luke

Has anyone looked at this patch yet? I'm assuming that since it was
written against 5.6.1 rather than the current bleadperl and it isn't in
diff -u format, a new patch might be needed.

One question comes to mind when I look at it, though. Have you tested
the patch on a non-Tiger system?

I haven't tested it on a non-Tiger system (I don't have access to
one).

Luke

--
Luke Closs
PureMessage Developer
There is always time to juggle in the Sophos Zone.

@p5pRT
Copy link
Author

p5pRT commented Sep 13, 2005

From @doughera88

On Mon, 12 Sep 2005, Luke Closs wrote​:

On Sat, Sep 10, 2005 at 10​:40​:25PM -0700, Steve Peters via RT wrote​:

Has anyone looked at this patch yet? I'm assuming that since it was
written against 5.6.1 rather than the current bleadperl and it isn't in
diff -u format, a new patch might be needed.

One question comes to mind when I look at it, though. Have you tested
the patch on a non-Tiger system?

I haven't tested it on a non-Tiger system (I don't have access to
one).

If Merijn doesn't beat me to it, I'll try to look at it in the next couple
of days.

--
  Andy Dougherty doughera@​lafayette.edu

@p5pRT
Copy link
Author

p5pRT commented Sep 13, 2005

From @Tux

On Tue, 13 Sep 2005 07​:52​:06 -0400 (EDT), Andy Dougherty
<doughera@​lafayette.edu> wrote​:

On Mon, 12 Sep 2005, Luke Closs wrote​:

On Sat, Sep 10, 2005 at 10​:40​:25PM -0700, Steve Peters via RT wrote​:

Has anyone looked at this patch yet? I'm assuming that since it was
written against 5.6.1 rather than the current bleadperl and it isn't in
diff -u format, a new patch might be needed.

One question comes to mind when I look at it, though. Have you tested
the patch on a non-Tiger system?

I haven't tested it on a non-Tiger system (I don't have access to
one).

If Merijn doesn't beat me to it, I'll try to look at it in the next couple
of days.

It was in my queue waiting for someone that could test on Tiger did so
It's a rather simple change, but as I don't have access to any MacOS like
OS's, I waited for comments

--
H.Merijn Brand Amsterdam Perl Mongers (http​://amsterdam.pm.org/)
using Perl 5.6.2, 5.8.0, 5.8.5, & 5.9.2 on HP-UX 10.20, 11.00 & 11.11,
AIX 4.3 & 5.2, SuSE 9.2 & 9.3, and Cygwin. http​://www.cmve.net/~merijn
Smoking perl​: http​://www.test-smoke.org, perl QA​: http​://qa.perl.org
reports to​: smokers-reports@​perl.org, perl-qa@​perl.org

@p5pRT
Copy link
Author

p5pRT commented Sep 13, 2005

@nwc10 - Status changed from 'open' to 'resolved'

@p5pRT p5pRT closed this as completed Sep 13, 2005
@p5pRT
Copy link
Author

p5pRT commented Sep 13, 2005

From @nwc10

On Fri, Jul 01, 2005 at 09​:56​:31PM -0000, Luke Closs wrote​:

# New Ticket Created by Luke Closs
# Please include the string​: [perl #36448]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl5/Ticket/Display.html?id=36448 >

Hello,

It looks like Apple has changed how ar works in xcode 2.1. As a
result, Configure thinks that ar can generate random libraries.

This breaks linking libperl.a, for instance when building PerlApp.

Configure doesn't currently allow overriding ranlib when Configure
detects that ranlib isn't necessary. My patch fixes this, and sets
ranlib for darwin.

Note that this patch is for ActivePerl 5.6.1 (it doesn't appear to be
fixed in 5.9.2), so it probably needs to be manually applied...

Indeed it did.

Thanks, applied (25402)

Sorry about the delay on this one.

Nicholas Clark

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant