[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[sr #110307] Upstream: netbsd/patch-lib_autoconf_fortran.m4

From: Remi Attab
Subject: [sr #110307] Upstream: netbsd/patch-lib_autoconf_fortran.m4
Date: Mon, 7 Sep 2020 14:34:39 -0400 (EDT)
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:80.0) Gecko/20100101 Firefox/80.0


                 Summary: Upstream: netbsd/patch-lib_autoconf_fortran.m4
                 Project: Autoconf
            Submitted by: rattab
            Submitted on: Mon 07 Sep 2020 06:34:38 PM UTC
                Category: None
                Priority: 5 - Normal
                Severity: 2 - Minor
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
        Operating System: *BSD



`Rpath is a linker option, so prefix it with -Wl. Bump revision.`

Patch can be merged as is but needs to be validated for correctness.

Comments from Zack W.
> The macro being changed is quite hairy and I think this is higher-risk than
it looks. -Wl, is a GCC feature (also supported by LLVM). It may be necessary
to do something cleverer here to make sure this still works with other Fortran
compilers.  Please file a bug for this one.  If you can dig up discussion of
why the change was made on the NetBSD mailing lists, commit history, etc. that
would be really helpful.

Wasn't able to find any mailing list threads related to this although I may
just be bad at tracking down information in mailing lists. Otherwise, will try
reaching out to the author and see if he can provide additional context.

Upstream patch triage project: https://git.sr.ht/~rattab/autoconf-upstream


File Attachments:

Date: Mon 07 Sep 2020 06:34:38 PM UTC  Name: patch-lib_autoconf_fortran.m4 
Size: 498B   By: rattab



Reply to this item at:


  Message sent via Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]