bazel icon indicating copy to clipboard operation
bazel copied to clipboard

Remove objc_library transition

Open keith opened this issue 2 years ago • 9 comments

keith avatar Nov 07 '23 17:11 keith

We're not quite ready to do this yet. Internally we still have this transition enabled.

googlewalt avatar Apr 09 '24 21:04 googlewalt

Update: We're still not quite ready to do this yet. Internally we still have this transition enabled. :)

comius avatar May 15 '24 13:05 comius

Is there a way to make this transition internal to Google then?

brentleyjones avatar May 15 '24 13:05 brentleyjones

Is there a way to make this transition internal to Google then?

It's possible. I could spend some time making the transition internal to Google, or towards doing cleanup.

How would that help Bazel? Because you wouldn't be able to flip the flag?

comius avatar May 16 '24 14:05 comius

I think the biggest benefit is it breaks the dependency on apple_support

keith avatar May 16 '24 15:05 keith

I think the biggest benefit is it breaks the dependency on apple_support

Ah, I missed that. I'm all for removing dependencies. Consider this PR approved, I'll do the import&changes necessary for internal world.

comius avatar May 17 '24 03:05 comius

On a side note, is there any objection that I make j2objc_library disappear?

comius avatar May 17 '24 04:05 comius

As far as we know there are no public users of the j2objc rules

keith avatar May 17 '24 05:05 keith

I rebased here since this was super out of date, hopefully that's helpful

keith avatar May 17 '24 17:05 keith

Thank you @comius!

brentleyjones avatar May 21 '24 16:05 brentleyjones

Is it possible to backport this to 7.3?

meteorcloudy avatar Jun 11 '24 08:06 meteorcloudy

Is it possible to backport this to 7.3?

I don't mind. It makes an incompatible flag a no-op, but the behaviour matches with the default settings of the flag.

comius avatar Jun 11 '24 08:06 comius

@bazel-io fork 7.3.0

meteorcloudy avatar Jun 11 '24 10:06 meteorcloudy