#39046 closed defect (invalid)
zsh-5.0.2 fails with clang
Reported by: | Ionic (Mihai Moldovan) | Owned by: | larryv (Lawrence Velázquez) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.1.3 |
Keywords: | haspatch | Cc: | |
Port: | zsh |
Description
zsh is currently failing to compile with clang due to it working around a bug in Apple's GCC.
Don't use this workaround when compiling with clang. Patch attached.
Attachments (3)
Change History (8)
Changed 12 years ago by Ionic (Mihai Moldovan)
Attachment: | zsh-portfile.diff added |
---|
Changed 12 years ago by Ionic (Mihai Moldovan)
Attachment: | patch-clang.diff added |
---|
Don't use the workaround when compiling with clang.
comment:1 Changed 12 years ago by Ionic (Mihai Moldovan)
Also sent upstream: http://www.zsh.org/mla/workers/2013/msg00449.html
comment:2 Changed 12 years ago by larryv (Lawrence Velázquez)
Cc: | larryv@… removed |
---|---|
Keywords: | haspatch added |
Owner: | changed from macports-tickets@… to larryv@… |
Status: | new → assigned |
Replying to ionic@…:
zsh is currently failing to compile with clang due to it working around a bug in Apple's GCC.
I just rebuilt zsh @5.0.2+universal, using clang-425.0.28. Please attach a log of your build failure.
Changed 12 years ago by Ionic (Mihai Moldovan)
Not using +universal though, but same compiler.
comment:3 follow-up: 4 Changed 12 years ago by Ionic (Mihai Moldovan)
I can confirm zsh as such is building fine with clang, but not with ccache+clang.
It's ccache messing up the source fed into clang.
comment:4 Changed 12 years ago by larryv (Lawrence Velázquez)
Resolution: | → invalid |
---|---|
Status: | assigned → closed |
Cool. The patch is probably fine for upstream, but I don’t like patching/reinplace
-ing/molesting the source more than is necessary.
comment:5 Changed 12 years ago by larryv (Lawrence Velázquez)
ccache does have a taste for mischief: http://petereisentraut.blogspot.com/2011/09/ccache-and-clang-part-2.html
Add patch to zsh's Portfile.