Skip to content

fix: restore missing listAttribute opener in ExportTestSuite.launch#1343

Merged
joaodinissf merged 1 commit into
dsldevkit:masterfrom
joaodinissf:fix/export-launch-malformed-xml
May 18, 2026
Merged

fix: restore missing listAttribute opener in ExportTestSuite.launch#1343
joaodinissf merged 1 commit into
dsldevkit:masterfrom
joaodinissf:fix/export-launch-malformed-xml

Conversation

@joaodinissf
Copy link
Copy Markdown
Collaborator

The opening <listAttribute key="org.eclipse.debug.core.MAPPED_RESOURCE_PATHS"> tag was dropped in 38918bc (feat: use junit5 platform in our tests), which replaced it with the new ATTR_FORCE_SYSTEM_CONSOLE_ENCODING attribute instead of inserting alongside. The following <listEntry> and </listAttribute> have been orphaned ever since, leaving the file malformed XML.

Eclipse PDE's launch-config parser was lenient enough to keep the launch usable, but Eclipse's XML language server now flags it as a parse error:

The element type "launchConfiguration" must be terminated by the matching end-tag "". [ETagRequired]

xmllint --noout confirms the file is well-formed after the fix; all other .launch files in the repo were already valid.

🤖 Generated with Claude Code

The opening <listAttribute key="org.eclipse.debug.core.MAPPED_RESOURCE_PATHS">
tag was dropped in 38918bc ("feat: use junit5 platform in our tests"),
which replaced it with the new ATTR_FORCE_SYSTEM_CONSOLE_ENCODING attribute
instead of inserting alongside. The following <listEntry> and </listAttribute>
have been orphaned ever since, leaving the file malformed XML.

Eclipse PDE's launch-config parser was lenient enough to keep the launch
usable, but Eclipse's XML language server flags it as a parse error.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@joaodinissf joaodinissf enabled auto-merge (rebase) May 16, 2026 20:56
@joaodinissf joaodinissf merged commit f54e079 into dsldevkit:master May 18, 2026
4 checks passed
@joaodinissf joaodinissf deleted the fix/export-launch-malformed-xml branch May 18, 2026 07:41
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.

2 participants