ProdriveTechnologies / bazel-latex

Bazel build system rules for LaTeX
Apache License 2.0
73 stars 35 forks source link

Remove tool wrapper #113

Closed solsjo closed 1 year ago

solsjo commented 1 year ago

@Kernald @patbro, when you have time. Or someone else with merge permissions. This chain of commits greatly simplifies rule logic and at the same time reduces build times of build targets.

patbro commented 1 year ago

Thanks for the trigger! I am still AFK, will probably get to this tomorrow.

solsjo commented 1 year ago

@patbro Thanks! :) No stress!

Kernald commented 1 year ago

I take it the first commit here is the one from #109?

solsjo commented 1 year ago

Yes!

patbro commented 1 year ago

@Kernald @patbro, when you have time. Or someone else with merge permissions. This chain of commits greatly simplifies rule logic and at the same time reduces build times of build targets.

I believe you know better what you're doing than me ;-) No comments whatsoever from my side, I will leave it up to @Kernald to merge it.

solsjo commented 1 year ago

Don't say that. My path to hubris is known to be short. :D Thanks for the review @patbro

solsjo commented 1 year ago

@Kernald , when you have time :)

solsjo commented 1 year ago

@Kernald, when you have time :)