Opened 14 years ago

Closed 14 years ago

#22660 closed defect (fixed)

Can't install pecl/mailparse with MacPorts' php5-mbstring

Reported by: andy@… Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version: 1.8.1
Keywords: Cc: nox@…
Port: php5-mbstring, php5-mailparse

Description

The PECL/mailparse library compiles against the mbstring extension, *including the headers* (not just the .so).

Reproduce

# port install php5-mbstring
# pecl install mailparse
-------
pecl/mailparse requires PHP extension "mbstring"
No valid packages found
install failed

When trying to compile manually, via phpize / make:

In file included from /private/tmp/pear/download/mailparse-2.1.5/mailparse.c:27:
/private/tmp/pear/download/mailparse-2.1.5/php_mailparse.h:100:49: error: ext/mbstring/libmbfl/mbfl/mbfilter.h: No such file or directory
In file included from /private/tmp/pear/download/mailparse-2.1.5/php_mailparse.h:109,
                 from /private/tmp/pear/download/mailparse-2.1.5/mailparse.c:27:
/private/tmp/pear/download/mailparse-2.1.5/php_mailparse_mime.h:68: error: expected specifier-qualifier-list before ‘mbfl_convert_filter’
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c: In function ‘zif_mailparse_determine_best_xfer_encoding’:
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:884: error: variable ‘bestenc’ has initializer but incomplete type
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:884: error: ‘mbfl_no_encoding_7bit’ undeclared (first use in this function)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:884: error: (Each undeclared identifier is reported only once
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:884: error: for each function it appears in.)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:884: error: storage size of ‘bestenc’ isn’t known
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:900: error: ‘mbfl_no_encoding_8bit’ undeclared (first use in this function)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:902: error: ‘mbfl_no_encoding_base64’ undeclared (first use in this function)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:912: error: ‘mbfl_no_encoding_qprint’ undeclared (first use in this function)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:915: warning: cast to pointer from integer of different size
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c: In function ‘zif_mailparse_stream_encode’:
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:950: error: storage size of ‘enc’ isn’t known
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:951: error: ‘mbfl_convert_filter’ undeclared (first use in this function)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:951: error: ‘conv’ undeclared (first use in this function)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:969: error: ‘mbfl_no_encoding_invalid’ undeclared (first use in this function)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:980: error: ‘mbfl_no_encoding_8bit’ undeclared (first use in this function)
/private/tmp/pear/download/mailparse-2.1.5/mailparse.c:988: error: ‘mbfl_no_encoding_qprint’ undeclared (first use in this function)

Solution

Allow mbstring to be compiled directly into PHP5 with --enable-mbstring. (Pre-php5extension behavior?)

Admittedly, mailparse is one of the few (only?) PECL packages that requires an extension in this manner, but it's useful enough.

Change History (5)

comment:1 Changed 14 years ago by jmroot (Joshua Root)

Cc: ryandesign@… andy@… removed
Milestone: MacPorts 1.8.2
Owner: changed from macports-tickets@… to ryandesign@…
Port: php5-mbstring added

comment:2 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Status: newassigned

comment:3 in reply to:  description ; Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

As of r61490, all php5extension ports that have headers (including php5-mbstring) now install them.

Next up: make a php5-mailparse port.

comment:4 in reply to:  3 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to ryandesign@…:

As of r61490, all php5extension ports that have headers (including php5-mbstring) now install them.

Make that as of r61501.

comment:5 in reply to:  3 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: nox@… added
Port: php5-mailparse added
Resolution: fixed
Status: assignedclosed

Replying to ryandesign@…:

Next up: make a php5-mailparse port.

Anthony was good enough to make this port for us, in r62551.

Note: See TracTickets for help on using tickets.