Opened 10 years ago

Closed 10 years ago

#44447 closed defect (fixed)

p5-eperl fails to build with perl5.16 (with patch)

Reported by: cfaerber@… Owned by: neverpanic (Clemens Lang)
Priority: Normal Milestone:
Component: ports Version: 2.3.1
Keywords: haspatch Cc: dimkab, kurthindenburg (Kurt Hindenburg)
Port: p5-eperl

Description

p5-eperl fails to build with perl5.16:

:info:build /usr/bin/clang -arch x86_64 -Wall -g -ggdb3 -O3 -pipe -Os  -fno-common -DPERL_DARWIN -I/opt/local/include -fno-strict-aliasing -fstack-protector -I/opt/local/include -I/opt/local/lib/perl5/5.16.3/darwin-thread-multi-2level/CORE -I. -c eperl_perl5.c
:info:build 1 warning generated.
:info:build 97 warnings generated.
:info:build 2 warnings generated.
:info:build In file included from eperl_perl5.c:38:
:info:build ./eperl_perl5.h:76:11: warning: 'PL_defoutgv' macro redefined
:info:build #  define PL_defoutgv defoutgv
:info:build           ^
:info:build /opt/local/lib/perl5/5.16.3/darwin-thread-multi-2level/CORE/embedvar.h:149:9: note: previous definition is here
:info:build #define PL_defoutgv             (vTHX->Idefoutgv)
:info:build         ^
:info:build eperl_perl5.c:84:19: error: use of undeclared identifier 'defoutgv'
:info:build     IoFLAGS(GvIOp(PL_defoutgv)) |= IOf_FLUSH; /* $|=1 */
:info:build                   ^

The patch file was originally from RedHat, and has also been included in Debian.

Attachments (2)

p5-eperl_perl5.16-patch.diff (1.6 KB) - added by cfaerber@… 10 years ago.
patch for p5-eperl with perl5.16
p5-eperl_perl5.16-patch.2.diff (1.2 KB) - added by cfaerber@… 10 years ago.
new patch without change of build revision

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by cfaerber@…

patch for p5-eperl with perl5.16

comment:1 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

Keywords: haspatch added
Owner: changed from macports-tickets@… to cal@…
Port: p5-eperl added

Is it only a build failure? If so, the revision should not be increased.

comment:2 Changed 10 years ago by cfaerber@…

Yes, it's just a build failure.

Changed 10 years ago by cfaerber@…

new patch without change of build revision

comment:3 Changed 10 years ago by dimkab

Cc: dima.batenkov@… added

Cc Me!

comment:4 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:5 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: newclosed

thanks r128941

Note: See TracTickets for help on using tickets.