Opened 12 years ago

Closed 8 years ago

Last modified 8 years ago

#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)

dircproxy_clang_patches.tgz (1.5 KB) - added by chris@… 12 years ago.
Patch files for dircproxy sources and to the Portfile

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by chris@…

Attachment: dircproxy_clang_patches.tgz added

Patch files for dircproxy sources and to the Portfile

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 Liondircproxy 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 8 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 8 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed

r142602 (maintainer timeout)

comment:6 Changed 8 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.

Note: See TracTickets for help on using tickets.