Bug 56113 - ports/Mk/bsd.openssl.mk has linker options in CFLAGS
Summary: ports/Mk/bsd.openssl.mk has linker options in CFLAGS
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Dirk Meyer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-08-29 03:00 UTC by Oliver Eikemeier
Modified: 2003-08-30 09:01 UTC (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Eikemeier 2003-08-29 03:00:25 UTC
When a port defines USE_OPENSSL, it gets
  CFLAGS+=		-Wl,-rpath,/usr/lib:${LOCALBASE}/lib
or
  CFLAGS+=		-Wl,-rpath,${LOCALBASE}/lib
(introduced in revision 1.4: http://www.freebsd.org/cgi/cvsweb.cgi/ports/Mk/bsd.openssl.mk.diff?r1=1.3&r2=1.4)


If this port just compiles *.c to *.o files and links them later,
the compiler emits two warning lines for every file compiled:
  cc: -rpath: linker input file unused since linking not done
  cc: /usr/lib:/usr/local/lib: linker input file unused since linking not done

This makes it hard to see other warnings that may be important.

Fix: 

Maybe the line can be simply removed or moved to LDFLAGS.
How-To-Repeat: 
Compile a port that defines USE_OPENSSL, i.e. port mail/ssmtp.
Comment 1 Kirill Ponomarev freebsd_committer freebsd_triage 2003-08-29 11:56:09 UTC
Responsible Changed
From-To: freebsd-ports-bugs->portmgr

portmgr territory
Comment 2 Joe Marcus Clarke freebsd_committer freebsd_triage 2003-08-29 13:58:23 UTC
Responsible Changed
From-To: portmgr->dinoex

Actually, dinoex, needs to look this over.
Comment 3 Dirk Meyer freebsd_committer freebsd_triage 2003-08-29 14:52:10 UTC
State Changed
From-To: open->analyzed


unfortunately from the 500 ports depends more then 100  
on the presence of this option to work as expected. 

I will make an extra option to supppress or add these.
Comment 4 Dirk Meyer freebsd_committer freebsd_triage 2003-08-30 09:01:06 UTC
State Changed
From-To: analyzed->closed

changes have been committed.