From: Arjen Markus <arjen.markus895@gmail.com>
To: Steve Kargl <sgk@troutmask.apl.washington.edu>
Cc: Fortran List <fortran@gcc.gnu.org>
Subject: Re: F2018 C937
Date: Fri, 20 Aug 2021 08:49:23 +0200 [thread overview]
Message-ID: <CAMCbSMoNxaHD_MfKKyDJoCdWoQpcFgg2zbHSLiS__cDHLQp-Kw@mail.gmail.com> (raw)
In-Reply-To: <20210820064651.GA28166@troutmask.apl.washington.edu>
Yes, I already had arranged for that copyright one and a half years ago,
but my first attempts failed (learning curve, real life getting in the way
etc.)
Op vr 20 aug. 2021 om 08:47 schreef Steve Kargl <
sgk@troutmask.apl.washington.edu>:
> Feel free to ask questions. I forgot to point to
> the contributing page
>
> https://gcc.gnu.org/contribute.html
>
> At one time, you need to assign copyright to the FSF.
> That is no longer required. Scan the above page for
> details.
>
> --
> steve
>
> On Fri, Aug 20, 2021 at 08:36:42AM +0200, Arjen Markus wrote:
> > Hi Steve,
> >
> > thanks for this detailed workflow. I am familiar enough with git to know
> > that there are myriads of procedures possible ;). Like you said, the
> first
> > three steps have been done. I will get working on step 4 and work my way
> > down the list.
> >
> > Regards,
> >
> > Arjen
> >
> > Op do 19 aug. 2021 om 18:23 schreef Steve Kargl <
> > sgk@troutmask.apl.washington.edu>:
> >
> > > Arjen,
> > >
> > > If this is your first go around with patching gfortran,
> > > I'll suggest running the testsuite (if you haven't, see
> > > step 5. below; if you have see step 6.). I suspect you
> > > already know much of what I enumerate below, but it may
> > > help others.
> > >
> > > With a bug report, the workflow for me is/was
> > >
> > > 1. Check versions of the standard to determine if it is
> > > a bug, and what is the expected result.
> > >
> > > 2. Identify where the problem can be addressed in source.
> > >
> > > For you (or anyone else interested in gfortran development),
> > > I suspect there a lot of question about how to do this
> > > step and the structure of the gfortran source code. I can
> > > answer some of those questions in follow-up emails. Send
> > > them to fortran@ and CC me.
> > >
> > > 3. Develop patch.
> > >
> > > I've done the first 3 steps. You are now at testing the patch.
> > >
> > > 4. Build gfortran with the patch. Assuming a Unix-like system,
> > > I have gcc/gccx with the source and gcc/objx is the build
> > > directory. So, for a first time build on an N cpu system do
> > >
> > > % cd gcc/objx
> > > % ../gcc/gccx/configure --prefix=$HOME/work/x \
> > > --enable-languages=c,c++,fortran,lto \
> > > --enable-bootstrap --disable-nls --enable-checking
> > > % make -j N-1 bootstrap && make install
> > >
> > > Otherwise,
> > >
> > > % cd gcc/objx
> > > % make -j N-1 && make install
> > >
> > > This installs everything in $HOME/work/x.
> > >
> > > 5. Run the testsuite to check for regressions. If any occur,
> > > fix regressions or fix the patch.
> > >
> > > % make -j N-1 check-fortran
> > > % tail gcc/testsuite/gfortran/gfortran.sum
> > >
> > > === gfortran Summary ===
> > >
> > > # of expected passes 58647
> > > # of expected failures 253
> > > # of unsupported tests 92
> > > objx/gcc/gfortran version 12.0.0 20210816 (experimental) (GCC)
> > >
> > > With the C937 and C949 patches, I changed "typespec" to
> > > "type-spec" in nearby unrelated error messages. Both are
> > > being used and type-spec matches the standard. This may
> > > cause a regression, so one or more testcases may need a change.
> > > The C937 patch did not cause a regression. The C949 one
> > > did. You'll see a line like
> > >
> > > # of unexpected failures 7
> > >
> > > The file gcc/testsuite/gfortran/gfortran.log contains the
> > > buildlog, which is huge. You can find the failures with
> > > a search for lines containing ^FAIL.
> > >
> > > 6. Prepare ChangeLog.
> > >
> > > This has changed with git so you'll need to ask Tobias,
> > > Thomas, Harald, or on the gcc@ list for guidance.
> > >
> > > 7. Submit patch to fortran@gcc and gcc-patches@gcc asking
> > > for review.
> > >
> > > 8. Wait a few days. Ping fortran@gcc and gcc-patches@gcc.
> > >
> > > 9. Wait a few days. Ping fortran@gcc and gcc-patches@gcc.
> > >
> > > As I developed, the original patch and presumably you reviewed
> > > it for correctness, you can probably skip step 9.
> > >
> > > 10. Wait a few days. Commit patch with or without a review.
> > >
> > > I do not know if you need any explicit access/permission to
> > > commit a patch. I iknow very little about git and how it
> > > works.
> > >
> > > At this point, you can either backport the patch to release
> > > branches or close the PR. For me, I always did a backport
> > > if it was a trivial task. At some point, HEAD will diverge
> > > sufficiently from a branch, I would then stop backporting.
> > >
> > > --
> > > steve
> > >
> > > On Thu, Aug 19, 2021 at 05:10:47PM +0200, Arjen Markus wrote:
> > > > I have applied the patches locally (take care to restore the tabs
> ;)).
> > > > Should I now commit these changes or is there a more formal procedure
> > > > involved?
> > > >
> > > > Regards,
> > > >
> > > > Arjen
> > > >
> > > > Op do 19 aug. 2021 om 08:59 schreef Arjen Markus <
> > > arjen.markus895@gmail.com
> > > > >:
> > > >
> > > > > Hi Steve,
> > > > >
> > > > > I am willing to take up this challenge ;), as well as the patch for
> > > C949.
> > > > > It would be my next attempt to get acquainted with the source code
> (a
> > > first
> > > > > step hopefully to actively contribute).
> > > > >
> > > > > Regards,
> > > > >
> > > > > Arjen
> > > > >
> > > > > Op di 17 aug. 2021 om 21:02 schreef Steve Kargl via Fortran <
> > > > > fortran@gcc.gnu.org>:
> > > > >
> > > > >> For those that might care, I draw your attention to
> > > > >>
> > > > >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101951
> > > > >>
> > > > >> Good opportunity for a lurker to step forward and
> > > > >> become a gfortran committer. Otherwise, this patch
> > > > >> will fester in bugzilla the dozen or so other patches
> > > > >> I've attached to PRs.
> > > > >>
> > > > >> --
> > > > >> Steve
> > > > >>
> > > > >
> > >
> > > --
> > > Steve
> > >
>
> --
> Steve
>
next prev parent reply other threads:[~2021-08-20 6:49 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-17 19:01 Steve Kargl
2021-08-19 6:59 ` Arjen Markus
2021-08-19 15:10 ` Arjen Markus
2021-08-19 16:23 ` Steve Kargl
2021-08-20 6:36 ` Arjen Markus
2021-08-20 6:46 ` Steve Kargl
2021-08-20 6:49 ` Arjen Markus [this message]
2021-08-23 17:18 ` Steve Kargl
2021-08-23 18:37 ` Arjen Markus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAMCbSMoNxaHD_MfKKyDJoCdWoQpcFgg2zbHSLiS__cDHLQp-Kw@mail.gmail.com \
--to=arjen.markus895@gmail.com \
--cc=fortran@gcc.gnu.org \
--cc=sgk@troutmask.apl.washington.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).