Bug 169523

Summary: [patch] x11/xscreensaver: remove forced -O0 from CFLAGS on amd64
Product: Ports & Packages Reporter: vsjcfm
Component: Individual Port(s)Assignee: Doug Barton <dougb>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch-Makefile none

Description vsjcfm 2012-06-28 17:00:25 UTC

How-To-Repeat: Update ports and take a look to x11/xscreensaver/Makefile
You will see there the following:

.if ${ARCH} == "amd64"
CFLAGS+=       -O0
.endif

There is no need to add "-O0" to CFLAGS on amd64. I've built xscreensaver successfully with "-O2 -fno-strict-aliasing -pipe -march=core2"
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2012-06-28 17:00:35 UTC
Responsible Changed
From-To: freebsd-ports-bugs->dougb

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Doug Barton freebsd_committer freebsd_triage 2012-07-02 08:25:09 UTC
State Changed
From-To: open->analyzed


I'll give this a shot, thanks.
Comment 3 Doug Barton freebsd_committer freebsd_triage 2012-07-02 11:04:46 UTC
State Changed
From-To: analyzed->closed


Committed, thanks!
Comment 4 dfilter service freebsd_committer freebsd_triage 2012-07-02 11:04:55 UTC
dougb       2012-07-02 10:04:41 UTC

  FreeBSD ports repository

  Modified files:
    x11/xscreensaver     Makefile 
  Log:
  Forcing -O0 for amd64 no longer seems relevant
  
  PR:             ports/169523
  Submitted by:   Sayetsky Anton <vsjcfm@gmail.com>
  
  Revision  Changes    Path
  1.124     +0 -4      ports/x11/xscreensaver/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"