geode-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (GEODE-18) Initial Code Cleanup
Date Thu, 08 Oct 2015 22:14:27 GMT


ASF GitHub Bot commented on GEODE-18:

GitHub user robertgeiger opened a pull request:

    GEODE-18 fixing existing header files

    Fix existing license header files and some minor constant string fixes for GEODE-18
    Minor string constant changes in:
    No code changes, only header changes. Reviewed large diff as diligently as possible to
insure only header changes exist. Ran build; not all tests work on my laptop but nothing different
in tests observed after changes. Remaining work is appx 900 files without any apparent header;
these are hard to automate as some have existing license headers that must be preserved.

You can merge this pull request into a Git repository by running:

    $ git pull develop

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #20
commit 5695e121e9a93ed55d46955d451feef2523f8e69
Author: Robert Geiger <rgeiger@ampoolrocket.local>
Date:   2015-10-08T21:09:15Z

    GEODE-18 : replace old License headers with appropriate Apache License in files. (1) apply
0001-Replace-Pivotal-License-Header-with-Apache-License.patch, fixing any issues;
               (2) apply 0002-Add-Missing-Apache-License-headers.patch, fixing any issues;
(3) automate (python) replacement of remaining easily identified headers, check;
               (4) hand fix any remaining files. still needs appropriate text
and one more sweep to detect any files without any header is needed.

commit 1ab9b68b924f580fd367fc15d5acb6caa7ccfda9
Author: Robert Geiger <rgeiger@ampoolrocket.local>
Date:   2015-10-08T21:41:23Z

    GEODE-18 : fix accidental mode changes on some files in previous checkin; no code change
at all, file mode change only


> Initial Code Cleanup
> --------------------
>                 Key: GEODE-18
>                 URL:
>             Project: Geode
>          Issue Type: Task
>          Components: general
>    Affects Versions: 1.0.0-incubating
>            Reporter: Niall Pemberton
>            Assignee: Dick Cavender
>             Fix For: 1.0.0-incubating
>         Attachments: 0001-Replace-Pivotal-License-Header-with-Apache-License.patch, 0002-Add-Missing-Apache-License-headers.patch,
geode-rat-report-2015-05-11.txt, geode-rat-report-2015-05-17.txt
> Please see the Incubator guide:
>     *
> Source & Notice File policy:
>     *
> Apache RAT is a tool that can help with source file headers:
>      *

This message was sent by Atlassian JIRA

View raw message