Request a pre-review (I guess): YETUS-142

[Allen Wittenauer <aw...@altiscale.com>]

Hi.

	I’ve been playing around for a solution to YETUS-142 for a while and I think I’m close.  Could someone take a look at the patch to see if I’ve missed anything obvious?  I have NOT committed this back to the YETUS-83 branch so it hasn’t had any live testing yet.  I’ll likely do that in the next day or so, but that would only cover some of the use cases. :/

Thanks.

Re: Request a pre-review (I guess): YETUS-142

[Chris Nauroth <cn...@hortonworks.com>]

I just commented.  I think it looks good overall, and I only thought of a
couple more cases around odd-ball file extensions I've seen in C++
projects.

If someone else more experienced with Gradle could review the gradle.sh
change, that would be ideal.  I don't have a lot of experience using
Gradle.

--Chris Nauroth




On 11/2/15, 3:37 PM, "Allen Wittenauer" <aw...@altiscale.com> wrote:

>
>Hi.
>
>	I¹ve been playing around for a solution to YETUS-142 for a while and I
>think I¹m close.  Could someone take a look at the patch to see if I¹ve
>missed anything obvious?  I have NOT committed this back to the YETUS-83
>branch so it hasn¹t had any live testing yet.  I¹ll likely do that in the
>next day or so, but that would only cover some of the use cases. :/
>
>Thanks.