New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #22660 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

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

Reported by: andy@… Owned by: ryandesign@…
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

comment:1 Changed 4 years ago by jmr@…

  • Cc ryandesign@…, andy@… removed
  • Owner changed from macports-tickets@… to ryandesign@…
  • Port set to php5-mbstring
  • Milestone MacPorts 1.8.2 deleted

comment:2 Changed 4 years ago by ryandesign@…

  • Status changed from new to assigned

comment:3 in reply to: ↑ description ; follow-ups: ↓ 4 ↓ 5 Changed 4 years ago by ryandesign@…

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 4 years ago by ryandesign@…

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 3 years ago by ryandesign@…

  • Cc nox@… added
  • Status changed from assigned to closed
  • Resolution set to fixed
  • Port changed from php5-mbstring to php5-mbstring, php5-mailparse

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.