Skip to content

lookup for generated files: support file names used by automake#45

Merged
theodevelop merged 5 commits into
theodevelop:devfrom
GitMensch:patch-1
Apr 17, 2026
Merged

lookup for generated files: support file names used by automake#45
theodevelop merged 5 commits into
theodevelop:devfrom
GitMensch:patch-1

Conversation

@GitMensch

@GitMensch GitMensch commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

automake uses uses a set of different extensions and "basename" files, see Automake manual, Yacc and Lex support and executes the yacc/lex programs with ylwrap to produce this result

Type of change

  • Bug fix
  • New feature
  • Refactor / internal improvement
  • Docs / config only

What does this PR do?

  • adds automake supported extensions to Bison recognition pattern
  • adds the generated filenames (basename + x) in the candidates (if that feels like "too much", then we need a two-step approach)
  • if not found in default directories uses a two-step approach to find the generated files somewhere in the workspace

Additional: enables a manual action run in any branch (I can split that from this PR, and I think the CI should possibly run in any branch on commit/push, no?)

Related issue

missing piece for #27

How to test manually

use an automake project like GnuCOBOL, from a release tarball (in which case bison/flex generated files won't even be re-generated, so no configure/make needed)

Checklist

  • npm run compile passes with no new errors -> as seen in CI
  • Tests added or updated (npx ts-node ...) -> not planned, but sounds possible
  • Manual test done in VS Code --> done with the CI generated vsix (works as long as source and generated are in the same folder)
  • CHANGELOG.md updated --> explicit not done as the feature is already described, but could be added
  • No unintended files staged (node_modules, .env, dist...)

automake uses uses a set of different extensions and "basename" files, see [Automake manual, Yacc and Lex support](https://www.gnu.org/software/automake/manual/html_node/Yacc-and-Lex.html) and executes the yacc/lex programs with ylwrap to produce this result
@GitMensch GitMensch marked this pull request as draft April 16, 2026 11:47
@theodevelop theodevelop marked this pull request as ready for review April 17, 2026 08:01
@theodevelop theodevelop merged commit 670fb74 into theodevelop:dev Apr 17, 2026
1 check passed
@theodevelop theodevelop added the bug Something isn't working label Apr 17, 2026
@theodevelop theodevelop linked an issue Apr 17, 2026 that may be closed by this pull request
@GitMensch GitMensch deleted the patch-1 branch April 17, 2026 08:26
@GitMensch

Copy link
Copy Markdown
Contributor Author

oh no, not even a squash merge, so I'm now forever doomed to be documented to checked in a bunch of intermediate versions ;-) [that was one of the reasons to mark it as a draft (and may still be adjusted when merging to main), the second part is that I can't get the "build folder" part to work, see https://github.com//issues/27#issuecomment-4260029735]

@theodevelop

Copy link
Copy Markdown
Owner

Oh no, I didn't notice it was still a draft! I thought there were just a few things to fix, which I did in fix/issue-27 — sorry about that, I'll be more careful next time. Everything will be available in the next release!

@theodevelop theodevelop mentioned this pull request Apr 20, 2026
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: #line-based navigation between source and generated files

2 participants