Skip to content

Conversation

@noraj
Copy link
Contributor

@noraj noraj commented Sep 18, 2025

  • including the license in the gem as MIT includes a copyright, it's unique to each gem
  • lib/seccomp-tools.rb requires lib/seccomp-tools/asm/asm.rb
    • which requires lib/seccomp-tools/asm/compiler.rb
      • which requires lib/seccomp-tools/asm/sasm.tab.rb
        • which requires racc.

So even if racc may be unused at runtime because it's transitively required from the root level, the dependency either need must be provided in the gemspec or stop being required from root level.
For example, tasks/sasm.rake could require 'seccomp-tools/asm/sasm.tab', require 'seccomp-tools/asm/compiler' or require 'seccomp-tools/asm/asm' or whatever needed.

@david942j
Copy link
Owner

Thanks for the contribution!

@david942j david942j merged commit 2e7fadd into david942j:master Oct 31, 2025
@noraj noraj deleted the patch-1 branch October 31, 2025 10:10
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