New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #25623 (closed defect: fixed)

Opened 3 years ago

Last modified 23 months ago

mutt-devel : cannot compile ssl

Reported by: erwan@… Owned by: simon@…
Priority: Normal Milestone:
Component: ports Version: 1.9.1
Keywords: Cc: bernhard@…, cobbe@…, dmitry@…, dports@…, derekathomas@…
Port: mutt-devel

Description

when upgrading mutt_devel from 1.5.20_2 to 1.5.20_3, it fail to compile mutt_ssl.c with following error

/usr/bin/gcc-4.0 -DPKGDATADIR=\"/opt/local/share/mutt\" -DSYSCONFDIR=\"/opt/local/etc\" -DBINDIR=\"/opt/local/bin\" -DMUTTLOCALEDIR=\"/opt/local/share/locale\" -DHAVE_CONFIG_H=1 -I.  -I. -I. -I./imap -I/opt/local/include -I/opt/local/include -Iintl -I/opt/local/include -I/opt/local/include -I/opt/local/include -I/opt/local/include -I/opt/local/include -I./intl  -O2 -arch i386 -MT mutt_ssl.o -MD -MP -MF .deps/mutt_ssl.Tpo -c -o mutt_ssl.o mutt_ssl.c
mutt_ssl.c: In function 'check_host':
mutt_ssl.c:655: error: 'STACK' undeclared (first use in this function)
mutt_ssl.c:655: error: (Each undeclared identifier is reported only once
mutt_ssl.c:655: error: for each function it appears in.)
mutt_ssl.c:655: error: 'subj_alt_names' undeclared (first use in this function)
mutt_ssl.c: In function 'ssl_cache_trusted_cert':
mutt_ssl.c:738: warning: assignment from incompatible pointer type
make[2]: *** [mutt_ssl.o] Error 1
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2


used variants are mutt-devel @1.5.20_2+compress+gpgme+headercache+imap+pop+sasl+smtp+ssl+tokyocabinet

Attachments

mutt-devel.patch (1.2 KB) - added by dmitry@… 3 years ago.
mutt-devel.2.patch (1.4 KB) - added by dmitry@… 3 years ago.
Same patch, but increment the revision in the portfile (no need to apply this one, if you've applied the previous one)

Change History

comment:1 Changed 3 years ago by erwan@…

  • Cc erwan@… added

Cc Me!

comment:2 Changed 3 years ago by bernhard@…

  • Cc bernhard@… added

Cc Me!

comment:3 Changed 3 years ago by jmr@…

  • Owner changed from macports-tickets@… to simon@…
  • Cc erwan@… removed

Please remember to cc the maintainer, and note that you do not need to be in cc when you are the reporter.

comment:5 Changed 3 years ago by erwan@…

I also upgradede openssl (0.9.8 -> 1.0.0) during this upgrade session. Might it be related ?

comment:6 Changed 3 years ago by cobbe@…

  • Cc cobbe@… added

Cc Me!

comment:7 Changed 3 years ago by mike@…

  • Cc mike@… added

Cc Me!

comment:8 Changed 3 years ago by mike@…

Fails on a fresh macports install too (mutt-devel +ssl, openssl 1.0.0).

$ port -vv install mutt-devel +ssl

...

/usr/bin/gcc-4.2 -pipe -O2 -arch x86_64 -L/opt/local/lib -arch x86_64 -L/opt/local/lib -L/opt/local/lib -L/opt/local/lib -o pgpewrap pgpewrap.o wcscasecmp.o -lintl -liconv mutt_ssl.c: In function 'check_host': mutt_ssl.c:655: error: 'STACK' undeclared (first use in this function) mutt_ssl.c:655: error: (Each undeclared identifier is reported only once mutt_ssl.c:655: error: for each function it appears in.) mutt_ssl.c:655: error: 'subj_alt_names' undeclared (first use in this function) mutt_ssl.c: In function 'ssl_cache_trusted_cert': mutt_ssl.c:738: warning: assignment from incompatible pointer type make[2]: * [mutt_ssl.o] Error 1 make[2]: * Waiting for unfinished jobs.... mv -f .deps/sha1.Tpo .deps/sha1.Po mv -f .deps/pgppubring.Tpo .deps/pgppubring.Po make[1]: * [all-recursive] Error 1 make: * [all] Error 2 shell command " cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_mail_mutt-devel/work/mutt-1.5.20" && /usr/bin/make -j4 all " returned error 2

comment:9 Changed 3 years ago by rbgarga@…

  • Cc rbgarga@… added

Cc Me!

comment:10 Changed 3 years ago by dmitry@…

Cc Me!

comment:11 Changed 3 years ago by dmitry@…

  • Cc dmitry@… added

Cc Me!

Changed 3 years ago by dmitry@…

comment:12 Changed 3 years ago by dmitry@…

Okay, I pulled the patch off of the freebsd ports mailing list

[0] http://www.freebsd.org/cgi/query-pr.cgi?pr=146261

I think this is more accurate, than the one listed above.

And just in case anyone is burning to apply this patch, here's how to do it:

$ cd /opt/local/var/macports/sources/rsync.macports.org/release/ports/mail/mutt-devel $ sudo patch -p1 < /path/to/patch $ sudo port install mutt-devel +ssl [+others]

At this point, it should notice that the Portfile has been incremented, and you should be good to go!

Wouldn't mind some feedback too ;)

comment:13 Changed 3 years ago by mike@…

Patch (and instructions!) worked for me. +1.

comment:14 Changed 3 years ago by rbgarga@…

Worked fine for me too, thanks!

comment:15 Changed 3 years ago by kuperman@…

I'll confirm that this patch works for me as well. I use

mutt-devel @1.5.20_3+compress+gdbm+headercache+imap+pop+sasl+smtp+ssl+trash (active)

BTW, you can simplify getting to the proper folder to apply the patch using the following instead of the first command:

cd `port dir mutt-devel`

I've found that particular port command quite handy when tweaking Portfiles.

Thanks for this fix!

comment:16 Changed 3 years ago by rbgarga@…

  • Cc rbgarga@… removed

Cc Me!

comment:17 Changed 3 years ago by erwan@…

Works for me also. Great job.

comment:18 Changed 3 years ago by mike@…

  • Cc mike@… removed

Cc Me!

comment:19 Changed 3 years ago by dports@…

  • Cc dports@… added

Cc Me!

Changed 3 years ago by dmitry@…

Same patch, but increment the revision in the portfile (no need to apply this one, if you've applied the previous one)

comment:20 Changed 3 years ago by dmitry@…

So i took another glance over my "patch" just to make sure I didn't mess anything up (it was 2AM for me). From my point of view everything looks good.

No need to thank me, this really is simply the fix from FreeBSD MLs... and for those of you who haven't actually looked in the patch i invite you to. You'll see that in line 43 the fix is just ONE line. I guess it shouldn't suprise anyone since all that happened is it got broken by a dependency.

Anyway I also increased the revision number (which i forgot before), so it's good to go now! All the devs have to do is apply it to the port system :)

comment:21 Changed 3 years ago by dports@…

  • Status changed from new to closed
  • Resolution set to fixed

Works for me too -- and now committed in r69763. Thanks for finding the patch!

comment:22 Changed 23 months ago by derekathomas@…

  • Cc derekathomas@… added

Cc Me!

Note: See TracTickets for help on using tickets.