[ 
https://issues.apache.org/jira/browse/OFBIZ-8251?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15498305#comment-15498305
 ] 

Taher Alkhateeb edited comment on OFBIZ-8251 at 9/17/16 6:18 AM:
-----------------------------------------------------------------

Attached is a simple and clean fix. Essentially, it's better to just use Xlint 
which is equal to Xlint:all to show everything and you can dig through it for 
whatever you want.

What this fix requires is only passing a project property called Xlint. So .. 
the syntax is ./gradlew -PXlint build

For the record, I'm not very convinced of adding both Xlint and OWASP to the 
master build.gradle. I am busy at the moment but I do wish to remove both (even 
though I helped in implementing both) because I was just trying to help Jacques 
to implement it properly


was (Author: taher):
Attached is a simple and clean fix. Essentially, it's better to just use Xlint 
which is equal to Xlint:all to show everything and you can dig through it for 
whatever you want.

What this fix requires is only passing a project property called Xlint. So .. 
the syntax is ./gradlew -PXlint build

For the record, I'm not very convinced of adding both Xlint and OWASP to the 
master build.gradle. I am buys at the moment but I do wish to remove both (even 
though I helped in implementing both) because I was just trying to help Jacques 
to implement it properly

> Create a Gradle task to use Xlint arguments when building
> ---------------------------------------------------------
>
>                 Key: OFBIZ-8251
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-8251
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Jacques Le Roux
>            Assignee: Jacques Le Roux
>            Priority: Minor
>             Fix For: Upcoming Branch
>
>         Attachments: OFBIZ-8251.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to