#34876 closed defect (fixed)
dircproxy 1.0.5 does not compile with clang/llvm 3.0 on OS X Lion
Reported by: | chris@… | Owned by: | darwin@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | clang haspatch | Cc: | ryandesign (Ryan Carsten Schmidt), jmroot (Joshua Root), nerdling (Jeremy Lavergne) |
Port: | dircproxy |
Description (last modified by ryandesign (Ryan Carsten Schmidt))
Attempting to build the dircproxy port results in the following:
make[2]: Entering directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_irc_dircproxy/dircproxy/work/dircproxy-1.0.5/src' /Developer/usr/bin/clang -DHAVE_CONFIG_H -I. -I. -I.. -DSYSCONFDIR="\"/opt/local/etc\"" -I/opt/local/include -pipe -O2 -arch x86_64 -c irc_server.c irc_server.c:344:69: warning: passing 'int *' to parameter of type 'socklen_t *' (aka 'unsigned int *') converts between pointers to integer types with different sign [-Wpointer-sign] if (!getsockname(p->server_sock, (struct sockaddr *)&sock_addr, &len)) { ^~~~ /usr/include/sys/socket.h:609:74: note: passing argument to parameter here int getsockname(int, struct sockaddr * __restrict, socklen_t * __restrict) ^ irc_server.c:703:52: error: expression is not assignable s = (l ? l->next : p->squelch_modes) = n; ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ irc_server.c:744:52: error: expression is not assignable s = (l ? l->next : p->squelch_modes) = n; ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ irc_server.c:1089:73: warning: passing 'int *' to parameter of type 'socklen_t *' (aka 'unsigned int *') converts between pointers to integer types with different sign [-Wpointer-sign] getsockname(p->client_sock, (struct sockaddr *)&vis_addr, &len)) { ^~~~ /usr/include/sys/socket.h:609:74: note: passing argument to parameter here int getsockname(int, struct sockaddr * __restrict, socklen_t * __restrict) ^ 2 warnings and 2 errors generated. make[2]: *** [irc_server.o] Error 1 make[2]: Leaving directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_irc_dircproxy/dircproxy/work/dircproxy-1.0.5/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_irc_dircproxy/dircproxy/work/dircproxy-1.0.5' make: *** [all-recursive-am] Error 2 make: Leaving directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_irc_dircproxy/dircproxy/work/dircproxy-1.0.5' Command failed: cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_irc_dircproxy/dircproxy/work/dircproxy-1.0.5" && /usr/bin/make -w all Exit code: 2
The attached patch fixes all such errors.
Attachments (1)
Change History (7)
Changed 12 years ago by chris@…
Attachment: | dircproxy_clang_patches.tgz added |
---|
comment:1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | ryandesign@… added |
---|---|
Description: | modified (diff) |
Keywords: | clang haspatch added; patch removed |
Owner: | changed from macports-tickets@… to darwin@… |
Summary: | [patch] dircproxy 1.0.5 does not compile with clang/llvm 3.0 on OS X Lion → dircproxy 1.0.5 does not compile with clang/llvm 3.0 on OS X Lion |
darwin at kastenbalg.org, does this look ok to commit? It compiles ok for me.
comment:2 Changed 12 years ago by jmroot (Joshua Root)
It's not clear why you would want a bitwise or in the cfgfile.c patch. The rest looks good though. (Per the C spec, a conditional expression does not yield an lvalue.)
comment:3 Changed 12 years ago by chris@…
As I recall, there was another compilation problem there that I could not hit until I fixed the conditional expressions, but I cannot now remember what it was.
comment:4 Changed 9 years ago by nerdling (Jeremy Lavergne)
Cc: | jmr@… snc@… added |
---|---|
Version: | 2.1.1 |
Upstream has not seen a commit since 2009 and my emails to the maintainer bounce.
Chris: do you want to become maintainer for these packages or should we instead consider deletion?
comment:5 Changed 9 years ago by jmroot (Joshua Root)
Resolution: | → fixed |
---|---|
Status: | new → closed |
r142602 (maintainer timeout)
comment:6 Changed 9 years ago by chris@…
I'm willing to fork it and put an active fork up on github or the like, to accept bug fixes and make them myself; however, I do not have the time right now to do any original feature work on it.
Patch files for dircproxy sources and to the Portfile