Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 26, 2025

Plan for Adding GNU Cobol Compiler Support

Based on PR #34, I need to:

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

continue working on proposed PR #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)


✨ 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 AI and others added 2 commits October 26, 2025 09:48
- Add CobolParser for parsing COBOL dependencies
- Create maven-native-gnucobol module with compiler and linker
- Update test files to use getRawCommandline() instead of deprecated getCommandline()
- Fix typos: 'defautl' -> 'default', 'extention' -> 'extension'
- Fix indentation issues (tabs -> spaces) to match code style
- Apply spotless formatting to all files
- Update .gitignore with additional entries
- All tests passing

Co-authored-by: slachiewicz <[email protected]>
- Add missing maven-native-test dependency to gnucobol module
- Fix CobolParser documentation: COPY statements, not #include
- Fix CobolParser logic: recognize 'copy' starting with 'c', not 'o'
- Update class documentation for GNUCOBOLCompiler and GNUCOBOLLinker
- All tests passing

Co-authored-by: slachiewicz <[email protected]>
Copilot AI changed the title [WIP] Add Cobol compiling feature through GNU Cobol compiler Add GNU Cobol compiler support Oct 26, 2025
Copilot AI requested a review from slachiewicz October 26, 2025 09:55
Copilot finished work on behalf of slachiewicz October 26, 2025 09:55
@slachiewicz
Copy link
Member

@copilot gha cicd fails

The GitHub Actions workflow was trying to activate a 'run-its' profile that
doesn't exist in the project, causing build warnings. Removed the -P run-its
flag from the mvn verify command to fix the CI failure.

Co-authored-by: slachiewicz <[email protected]>
Copy link
Contributor Author

Copilot AI commented Oct 27, 2025

@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 finished work on behalf of slachiewicz October 27, 2025 02:10
@slachiewicz
Copy link
Member

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

Copy link
Contributor Author

Copilot AI commented Oct 28, 2025

@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 finished work on behalf of slachiewicz October 28, 2025 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Cobol compiling feature thru the GNU Cobol compiler

2 participants