Bug 159250 - [patch] multimedia/mplayer: clang detection fails
Summary: [patch] multimedia/mplayer: clang detection fails
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: Rene Ladan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-28 09:00 UTC by Test Rat
Modified: 2011-08-09 07:40 UTC (History)
0 users

See Also:


Attachments
a.diff (876 bytes, patch)
2011-07-28 09:00 UTC, Test Rat
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Test Rat 2011-07-28 09:00:25 UTC
In clang case configure script expects the first word to contain
compiler name. However, unlike lang/clang the one from base
embeds "FreeBSD" word before "clang".

How-To-Repeat: $ clang -v
FreeBSD clang version 3.0 (trunk 135360) 20110717
Target: x86_64-unknown-freebsd9.0
Thread model: posix

$ LOCALBASE/bin/clang -v
clang version 2.9 (tags/RELEASE_29/final)
Target: x86_64-portbld-freebsd9.0
Thread model: posix
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2011-07-28 09:00:34 UTC
Maintainer of multimedia/mplayer,

Please note that PR ports/159250 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/159250

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2011-07-28 09:00:37 UTC
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 3 Rene Ladan freebsd_committer freebsd_triage 2011-07-31 10:15:58 UTC
Responsible Changed
From-To: freebsd-ports-bugs->rene

Grab
Comment 4 Thomas E. Zander 2011-08-07 15:42:06 UTC
Due to the unfortunate fact that I don't have a FreeBSD box connected
to the internet right now, I can't test whether this patch works as
intended without side effects. It would be good to get confirmation
from some more testers that it does not break other compiler
configurations.
Comment 5 Rene Ladan freebsd_committer freebsd_triage 2011-08-09 07:39:30 UTC
State Changed
From-To: feedback->closed

Committed, thanks
Comment 6 dfilter service freebsd_committer freebsd_triage 2011-08-09 07:39:33 UTC
rene        2011-08-09 06:39:25 UTC

  FreeBSD ports repository

  Modified files:
    multimedia/mplayer/files patch-configure 
  Log:
  Fix build with clang: in contrast to lang/clang, the one from base adds
  "FreeBSD" in front of it which broke the detection of the compiler name.
  If the operating system name is added, use the second word as the compiler
  name.
  
  PR:             ports/159250
  Submitted by:   Test Rat (ttsestt gmail.com)
  Approved by:    maintainer (in private mail)
  
  Revision  Changes    Path
  1.7       +9 -0      ports/multimedia/mplayer/files/patch-configure
_______________________________________________
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"