maven-native icon indicating copy to clipboard operation
maven-native copied to clipboard

Add GNU Cobol compiler support

Open Copilot opened this issue 2 months ago • 4 comments

Plan for Adding GNU Cobol Compiler Support

Based on PR #34, I need to:

  • [x] Add CobolParser.java to maven-native-api for parsing COBOL dependencies
  • [x] Update PostE.java to support CobolParser with proper indentation (fix tabs)
  • [x] Create maven-native-gnucobol module with:
    • [x] AbstractCobolCompiler.java
    • [x] GNUCOBOLCompiler.java
    • [x] GNUCOBOLLinker.java (fix typo: "extention" -> "extension")
    • [x] Test files (fix typo: "defautl" -> "default" in CLinkerTest.java)
    • [x] CobolCompilerTest.java
    • [x] pom.xml and LICENSE.txt
  • [x] Update maven-native-components/pom.xml to include gnucobol module
  • [x] Update root pom.xml with dependency management
  • [x] Update existing test files to use getRawCommandline() instead of getCommandline() (fix indentation issues)
  • [x] Update .gitignore with additional entries
  • [x] Build and test the changes
  • [x] Run code formatting checks (spotless applied)
  • [x] Address code review feedback
    • [x] Add missing maven-native-test dependency
    • [x] Fix CobolParser documentation and logic
    • [x] Update class documentation for COBOL-specific classes
  • [x] Fix CI build failure
    • [x] Remove non-existent run-its profile from GitHub Actions workflow

All tasks completed successfully. The GNU Cobol compiler support is fully integrated with all issues resolved and CI fixed.

Original prompt

This section details on the original issue you should resolve

<issue_title>Add Cobol compiling feature thru the GNU Cobol compiler</issue_title> <issue_description>Followup https://github.com/mojohaus/maven-native/pull/34

continue working on proposed PR mojohaus/maven-native#34 - address review comments, rebase code on top of master. fix spotless formatting</issue_description>

Comments on the Issue (you are @copilot in this section)

  • Fixes mojohaus/maven-native#152

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot avatar Oct 26 '25 09:10 Copilot

@copilot gha cicd fails

slachiewicz avatar Oct 27 '25 02:10 slachiewicz

@copilot gha cicd fails

Fixed in commit 17ea735. The workflow was using a non-existent run-its profile. Removed the -P run-its flag from the build command.

Copilot avatar Oct 27 '25 02:10 Copilot

@copilot rebase on master to incorporate latest windows build fixes.

slachiewicz avatar Oct 28 '25 22:10 slachiewicz

@copilot rebase on master to incorporate latest windows build fixes.

The branch is already rebased on the latest master (68dea67), which includes the Windows build fixes from commit a3ba8b8. Build verified successfully with all tests passing.

Copilot avatar Oct 28 '25 22:10 Copilot