FR: Way to exclude jar from gazelle resolution

[bazel-contrib]

FR: Way to exclude jar from gazelle resolution

[chrismgrayftsinc]

Yeah I can probably do that -- no timeline though.

[illicitonion]

Sounds very sensible - can I interest you in putting together a PR?

This is where we parse out the metadata of jar files:

rules_jvm/java/gazelle/private/maven/resolver.go

Lines 24 to 51 in 087e403

	func NewResolver(installFile string, logger zerolog.Logger) (Resolver, error) {
	r := resolver{
	data: multiset.NewStringMultiSet(),
	logger: logger.With().Str("_c", "maven-resolver").Logger(),
	}
	c, err := loadConfiguration(installFile)
	if err != nil {
	r.logger.Warn().Err(err).Msg("not loading maven dependencies")
	return &r, nil
	}
	r.logger.Debug().Int("count", len(c.DependencyTree.Dependencies)).Msg("Dependency count")
	r.logger.Debug().Str("conflicts", fmt.Sprintf("%#v", c.DependencyTree.ConflictResolution)).Msg("Maven install conflict")
	for _, dep := range c.DependencyTree.Dependencies {
	for _, pkg := range dep.Packages {
	c, err := ParseCoordinate(dep.Coord)
	if err != nil {
	return nil, fmt.Errorf("failed to parse coordinate %v: %w", dep.Coord, err)
	}
	l := label.New("maven", "", bazel.CleanupLabel(c.ArtifactString()))
	r.data.Add(pkg, l.String())
	}
	}
	return &r, nil
	}

And this is where we parse out directives:

rules_jvm/java/gazelle/configure.go

Lines 74 to 99 in 087e403

	for _, d := range f.Directives {
	switch d.Key {
	case javaconfig.JavaExtensionDirective:
	switch d.Value {
	case "enabled":
	cfg.SetExtensionEnabled(true)
	case "disabled":
	cfg.SetExtensionEnabled(false)
	default:
	jc.lang.logger.Fatal().Msgf("invalid value for directive %q: %s: possible values are enabled/disabled",
	javaconfig.JavaExtensionDirective, d.Value)
	}
	case javaconfig.MavenInstallFile:
	cfg.SetMavenInstallFile(d.Value)
	case javaconfig.ModuleGranularityDirective:
	if err := cfg.SetModuleGranularity(d.Value); err != nil {
	jc.lang.logger.Fatal().Err(err).Msgf("invalid value for directive %q", javaconfig.ModuleGranularityDirective)
	}
	case javaconfig.TestMode:
	cfg.SetTestMode(d.Value)
	}
	}
	}

Wiring things up between the two should be pretty easy :)