-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Custom scanner implementation for Dialog Tag Language #174
Conversation
Signed-off-by: Dennis Behm <dennis.behm@de.ibm.com>
Signed-off-by: Dennis Behm <dennis.behm@de.ibm.com>
Signed-off-by: Dennis Behm <dennis.behm@de.ibm.com>
Signed-off-by: Dennis Behm <dennis.behm@de.ibm.com>
This PR has been validated by N. Latif and is (finally) ready for a review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few comments, otherwise it looks OK.
Should the README be revised too?
Scanners/README.md
Outdated
* Clone the IBM/DBB repository into IDZ/Eclipse. | ||
* Import the scanner implementation as a JAVA project | ||
* Download the `<DBB-HOME>/lib/dbb.core_2.0.x.x.jar` package from your IBM DBB toolkit installation to your development environment | ||
* Fix the build setup and of the scanner project. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small typo here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix in 2adfcc5
Scanners/README.md
Outdated
|
||
* Clone the IBM/DBB repository into IDZ/Eclipse. | ||
* Import the scanner implementation as a JAVA project | ||
* Download the `<DBB-HOME>/lib/dbb.core_2.0.x.x.jar` package from your IBM DBB toolkit installation to your development environment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should explicitly say "from your IBM DBB toolkit installation on z/OS Unix System Services to..."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix in 2adfcc5
Scanners/README.md
Outdated
* Select the previously created runner configuration | ||
* Select the export destination | ||
* In the Library handling section, select to `Copy required libraries into a sub-folder next to the generated JAR` option | ||
* Upload the generated JAR to z/OS Unix system services |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Copy the generated JAR to z/OS Unix system services as binary"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix in 2adfcc5
.project
Outdated
<?xml version="1.0" encoding="UTF-8"?> | ||
<projectDescription> | ||
<name>dbb</name> | ||
<comment></comment> | ||
<projects> | ||
</projects> | ||
<buildSpec> | ||
</buildSpec> | ||
<natures> | ||
</natures> | ||
</projectDescription> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to have a .project for the whole dbb repository?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix in 2adfcc5
Signed-off-by: Dennis Behm <dennis.behm@de.ibm.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I missed one typo in my previous review.
Scanners/README.md
Outdated
|
||
Sample | Description | ||
--- | --- | ||
[DTL Scanner](DTL%20Scanner) | A custom scanner implementation for z/OS ISPF Dialog Tag Language to detect it's dependencies. zAppBuild contains the language script for DTL. The scanner implementation leverages regex patterns to identify the referenced include files. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small typo here: it's dependencies
--> its dependencies
Sorry I missed that.
Signed-off-by: Dennis Behm <dennis.behm@de.ibm.com>
Signed-off-by: Dennis Behm <dennis.behm@de.ibm.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you!
Signed-off-by: Dennis Behm <dennis.behm@de.ibm.com>
DBB has an extensibility mechanism to contribute your own dependency scanner for languages that either is not covered by the DBB product itself or that you might use some custom syntax, that is custom to your environment.
This contribution from @campagboy is outlining how to build a customer scanner using the AbstractDependencyScanner API of DBB for the Dialog Tag Language in ISPF.
The DTL scanner comes with the java project, that you need to build with then add to the JAVA classpath.
Also combining the default DependencyScanner is possible with your own parsing logic is possible by writing a new scanner that invokes the default DependencyScanner then adds your own dependencies.