Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c62b27a7 authored by Alix Espino's avatar Alix Espino Committed by Gerrit Code Review
Browse files

Merge "Mark android_* modules with java_resources as unconvertible" into main

parents 596ee5c8 ee51bd65
Loading
Loading
Loading
Loading
+3 −0
Original line number Diff line number Diff line
@@ -1352,6 +1352,9 @@ func (a *AndroidLibrary) ConvertWithBp2build(ctx android.Bp2buildMutatorContext)
	if !supported {
		return
	}
	if hasJavaResources := aaptAttrs.ConvertJavaResources(ctx, commonAttrs); hasJavaResources {
		return
	}
	ctx.CreateBazelTargetModule(
		props,
		android.CommonAttributes{Name: name},
+13 −0
Original line number Diff line number Diff line
@@ -1638,6 +1638,15 @@ type bazelAndroidAppAttributes struct {
	Updatable        *bool
}

func (b bazelAapt) ConvertJavaResources(ctx android.Bp2buildMutatorContext, javaAttrs *javaCommonAttributes) bool {
	// TODO (b/300470246) bp2build support for java_resources & java_resource_dirs in android rules
	hasJavaResources := !javaAttrs.javaResourcesAttributes.Resources.IsEmpty()
	if hasJavaResources {
		ctx.MarkBp2buildUnconvertible(bp2build_metrics_proto.UnconvertedReasonType_UNSUPPORTED, "(b/300470246) java resources in android_* module")
	}
	return hasJavaResources
}

func convertWithBp2build(ctx android.Bp2buildMutatorContext, a *AndroidApp) (bool, android.CommonAttributes, *bazelAndroidAppAttributes) {
	aapt, supported := a.convertAaptAttrsWithBp2Build(ctx)
	if !supported {
@@ -1712,6 +1721,10 @@ func convertWithBp2build(ctx android.Bp2buildMutatorContext, a *AndroidApp) (boo
	if !supported {
		return false, android.CommonAttributes{}, &bazelAndroidAppAttributes{}
	}
	if hasJavaResources := aapt.ConvertJavaResources(ctx, commonAttrs); hasJavaResources {
		return false, android.CommonAttributes{}, &bazelAndroidAppAttributes{}
	}

	depLabels := bp2BuildInfo.DepLabels

	deps := depLabels.Deps