Bug 169523 - [patch] x11/xscreensaver: remove forced -O0 from CFLAGS on amd64
Summary: [patch] x11/xscreensaver: remove forced -O0 from CFLAGS on amd64
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: Doug Barton
Depends on:
Reported: 2012-06-28 17:00 UTC by vsjcfm
Modified: 2012-07-02 11:10 UTC (History)
0 users

See Also:

patch-Makefile (359 bytes, text/plain)
2012-06-28 17:00 UTC, vsjcfm
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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

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 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 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 2012-07-02 11:04:46 UTC
State Changed
From-To: analyzed->closed

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

  FreeBSD ports repository

  Modified files:
    x11/xscreensaver     Makefile 
  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
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"