Bug 155906 - lang/p5-F77 updated to 1.17; +Fix the broken port
Summary: lang/p5-F77 updated to 1.17; +Fix the broken port
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: freebsd-perl (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-24 06:40 UTC by lichray
Modified: 2011-05-05 09:34 UTC (History)
0 users

See Also:


Attachments
patch-p5-F77-1.17 (4.40 KB, text/plain)
2011-03-24 06:40 UTC, lichray
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description lichray 2011-03-24 06:40:01 UTC
	ExtUtils::F77 has been updated to 1.17. Check http://search.cpan.org/~kgb/ExtUtils-F77/F77.pm for details.

	Note: the old version 1.16_1 in ports does not work! The problems are:
	1. It only searches for g77-34 on FreeBSD;
	2. It does not set the libg2c path (since this lib has been removed from FreeBSD base system).

Fix: So here is what my patch does:
	1. Detects both g77-34 and gfortran;
	2. Uses a post-patch to set the CFLAGS and gfortran version correctly.
	3. Marks gfortran as the default compiler (since g77 is deprecated in ports tree), and enables g77-34 as a fallback.
	Now this port works whenever you have a fortran compiler installed from ports/packages, and obey ${FC} and ${CFLAGS} :)
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2011-03-24 06:40:12 UTC
Responsible Changed
From-To: freebsd-ports-bugs->perl

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 lichray 2011-03-24 12:16:39 UTC
Sorry... this patch massed something up. For these 2 lines:

++$F77config{Freebsd}{G77}{Cflags} = '-O2 -pipe -march=nocona
-fno-strict-aliasing';
++$F77config{Freebsd}{GFortran}{Cflags}   = '-O2 -pipe -march=nocona
-fno-strict-aliasing';

Please change these two Perl strings into '-O2', then the post-patch
will replace them with correct CFLAGS.

-- 
Zhihao Yuan
The best way to predict the future is to invent it.
Comment 3 dfilter service freebsd_committer freebsd_triage 2011-05-05 09:22:46 UTC
tota        2011-05-05 08:22:29 UTC

  FreeBSD ports repository

  Modified files:
    lang/p5-F77          Makefile distinfo pkg-plist 
  Added files:
    lang/p5-F77/files    patch-F77.pm 
  Log:
  - Update to 1.17
    * Detect both g77-34 and gfortran
    * Use a post-patch to set the CFLAGS and gfortran version correctly
    * Mark gfortran as the default compiler (since g77 is deprecated in ports
      tree), and enable g77-34 as a fallback
  
  PR:             ports/155906
  Submitted by:   Zhihao Yuan <lichray_AT_gmail_DOT_com>
  
  Revision  Changes    Path
  1.18      +6 -6      ports/lang/p5-F77/Makefile
  1.8       +2 -3      ports/lang/p5-F77/distinfo
  1.1       +65 -0     ports/lang/p5-F77/files/patch-F77.pm (new)
  1.6       +0 -1      ports/lang/p5-F77/pkg-plist
_______________________________________________
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"
Comment 4 TAKATSU Tomonari freebsd_committer freebsd_triage 2011-05-05 09:33:33 UTC
State Changed
From-To: open->closed

Committed, with minor changes. Thanks!