RFR(xxs): 8227170: (.hg)Ignore the JTwork and JTreport directories generated at the root of the repo
jai.forums2013 at gmail.com
Wed Jul 10 11:48:34 UTC 2019
Ping. Anyone willing to review and sponsor this change?
On 03/07/19 5:43 PM, Jaikiran Pai wrote:
> Can I please get a review and a sponsor for the patch for the
> enhancement noted in https://bugs.openjdk.java.net/browse/JDK-8227170?
> The webrev containing the patch is here
> For some context about this change - there's been some discussion on
> this in the jdk-dev list here
> The patch in the webrev, does the following things:
> - In addition to ignoring the JTwork and JTreport sub-directories, this
> now also ignores these directories at the root of the repo. This patch
> uses "syntax: glob" to easily ignore such directories, instead of the
> (default) "regex" syntax. In the jdk-dev mailing list discussion,
> Gustavo suggested to continue using the regex syntax and listing the
> following in the .hgignore:
> But that would then end up ignoring directories of the form
> "foobarJTreport" and files under it like "foobarJTreport/blah.txt". I
> guess we can still stick with a regex syntax and come up with a regex
> which ignores these directories both at the root as well as
> sub-directories, maybe something like:
> but IMO, that appears a bit too complex when compared to the glob syntax
> that's used in the webrev.
> - The other change in this webrev is to ignore the
> src/utils/hsdis/build/ directory which gets generated when
> src/utils/hsdis is built. I included this in this patch based on
> Gustavo's suggestion in that linked thread. I verified that this is
> indeed the "build" location used by hsdis tool, by checking the
> src/utils/hsdis/Makefile as well src/utils/hsdis/README.
> Gustavo also had a suggestion that we ignore the "oprofile_data"
> directory/files. But I don't have knowledge of what those files are and
> didn't have a way to ascertain the right location/rule to add to the
> .hgignore. So I have not included that in the current version of this
> webrev. I'm however open to including this in the webrev if Gustavo and
> others feel that's OK/needed.
More information about the build-dev