Patch for bug 47030
Steve Kargl
sgk@troutmask.apl.washington.edu
Sun Jan 6 17:42:00 GMT 2019
On Sun, Jan 06, 2019 at 09:02:31AM -0800, Marco Atzeri wrote:
> Am 06.01.2019 um 07:19 schrieb Thomas Koenig:
> > Hi Marco,
> >
> >> The copyright assignment was completed, and I uploaded a
> >> updated version of the patch
> >>
> >> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45348
> >>
> >> that passes the check_GNU_style.sh verification.
> >
> > Excellent!
> >
> > Can you submit the patch using the normal procedure,
> > with a ChangeLog entry and a test case?
> >
> > Regards
> >
> > Thomas
>
> What you mean as test case ?
>
> I put a test in the bug entry as attachment
> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44950
> but I have no idea how to implement formally for
> the build
>
Marco,
Add the testcase(s) to the gcc/testsuite/gfortran.dg directory.
Either use a descriptive name for the testcase(s) or PR number.
For example, for this PR you may have
gcc/testsuite/gfortran.dg/pr44950_1.f90
gcc/testsuite/gfortran.dg/pr44950_2.f90
The number of testcases depends on what your patch does. Here,
pr44950_1.f90 may test that error messages are correctly issued;
while pr44950_2.f90 tests that gfortran produces the correct
result.
I haven't looked at your patch, but suppose you changed
gcc/fortran/resolve.c. You create your diff by
svn add gcc/testsuite/gfortran.dg/pr44950_1.f90
svn add gcc/testsuite/gfortran.dg/pr44950_2.f90
svn diff gcc/fortran/resolve.c \
gcc/testsuite/gfortran.dg/pr44950_1.f90 \
gcc/testsuite/gfortran.dg/pr44950_2.f90 > pr44950.diff
In a separate text file, create the ChangeLog entries that
will be add to gcc/fortran/ChangeLog and gcc/testsuite/ChangeLog
when your patch is actually committed. ChangeLog entries have
the form
2019-01-06 Your Name <your_email_address>
<blank line>
<tab>PR fortran/44950
<tab>* resolve.c (name of function changed): What changed here (be brief).
(2nd function changed): What changed here.
2019-01-06 Your Name <your_email_address>
<blank line>
<tab>PR fortran/44950
<tab>* gfortran.dg/pr44950_1.f90: New test.
<tab>* gfortran.dg/pr44950_2.f90: Ditto.
Now, send email to fortran@gcc.gnu.org and gcc-patches@gcc.gnu.org
with the ChangeLog entries in the body of the email, tell us how
you tested the patch, and include the diff as an attachment.
See https://gcc.gnu.org/ml/fortran/2019-01/msg00015.html
--
Steve
More information about the Fortran
mailing list