Skip to content

h2xs is broken #8165

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

Closed
p5pRT opened this issue Oct 25, 2005 · 4 comments
Closed

h2xs is broken #8165

p5pRT opened this issue Oct 25, 2005 · 4 comments

Comments

@p5pRT
Copy link

p5pRT commented Oct 25, 2005

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

Searchable as RT37525$

@p5pRT
Copy link
Author

p5pRT commented Oct 25, 2005

From [email protected]

Hi,

You have a bug in the parameter parsing stage of h2xs, it aborts if
any [extra_libraries] are specified, because you have this line of
code in there​:-

GetOptions(%options) || usage;

but "-l" is not dealt with until later in your code. Changing the
above to this fixes the problem​:-

GetOptions(%options); # || usage;

FYI​:-

SYNOPSIS
  h2xs [OPTIONS ...] [headerfile ... [extra_libraries]]

EXAMPLES
  h2xs rpcsvc/rusers -L/opt/net/lib -lrpc

# h2xs rpcsvc/rusers -L/opt/net/lib -lrpc
Unknown option​: L/opt/net/lib
Unknown option​: lrpc
h2xs [OPTIONS ... ] [headerfile [extra_libraries]]
version​: 1.22
[snip]
extra_libraries
  are any libraries that might be needed for loading the
  extension, e.g. -lm would try to link in the math library.
#

(ends, doing nothing)

Kind Regards,
Chris Drake

@p5pRT
Copy link
Author

p5pRT commented Nov 11, 2005

From @smpeters

[christopher - Mon Oct 24 23​:49​:35 2005]​:

Hi,

You have a bug in the parameter parsing stage of h2xs, it aborts if
any [extra_libraries] are specified, because you have this line of
code in there​:-

GetOptions(%options) || usage;

but "-l" is not dealt with until later in your code. Changing the
above to this fixes the problem​:-

GetOptions(%options); # || usage;

FYI​:-

SYNOPSIS
h2xs [OPTIONS ...] [headerfile ... [extra_libraries]]

EXAMPLES
h2xs rpcsvc/rusers -L/opt/net/lib -lrpc

# h2xs rpcsvc/rusers -L/opt/net/lib -lrpc
Unknown option​: L/opt/net/lib
Unknown option​: lrpc
h2xs [OPTIONS ... ] [headerfile [extra_libraries]]
version​: 1.22
[snip]
extra_libraries
are any libraries that might be needed for loading the
extension, e.g. -lm would try to link in the math library.
#

(ends, doing nothing)

h2xs was broken.

h2xs rpcsvc/rusers -L/opt/net/lib -lrpc
Defaulting to backwards compatibility with perl 5.8.6
If you intend this module to be compatible with earlier perl versions,
please
specify a minimum perl version with the -b option.

Writing Rusers/ppport.h
Writing Rusers/lib/Rusers.pm
Writing Rusers/Rusers.xs
Writing Rusers/fallback/const-c.inc
Writing Rusers/fallback/const-xs.inc
Writing Rusers/Makefile.PL
Writing Rusers/README
Writing Rusers/t/Rusers.t
Writing Rusers/Changes
Writing Rusers/MANIFEST

h2xs -h
h2xs [OPTIONS ... ] [headerfile [extra_libraries]]
version​: 1.9

It looks like you have a very old version of h2xs, and the problems
you're seeing have since been fixed.

@p5pRT
Copy link
Author

p5pRT commented Nov 11, 2005

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

@p5pRT
Copy link
Author

p5pRT commented Nov 11, 2005

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

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