zdharma-continuum / zinit

🌻 Flexible and fast ZSH plugin manager
MIT License
2.74k stars 122 forks source link

A new ice build'' that's a shorthand for: null configure'' make'' #548

Closed psprint closed 5 months ago

psprint commented 11 months ago

Description

I think that the PR is important so I'm refreshing it (old: #458).

After the configure'' ice has been added the building of programs from sources became simpler. The new ice build'' is one step further – zinit null configure make for … will be equivalent of zinit build for …. The new ice is just a "pull-in" ice, as all that it does is to pull-in the other 3 ices. null ice pulling can be blocked with nonull. Parameters given to build'' are passed to the underlying make'' ice.

Motivation and Context

The motivation is to type-less and to have a clear way of compiling apps with zinit. Withbuild'' ice there's a single method of building all kinds of software, from autotools to meson. I also add a test case and update README.md.

Usage examples

zi id-as'slang' extract'!' build \
            for http://www.jedsoft.org/snapshots/slang-pre2.3.4-5.tar.gz

zi id-as from"gitlab.matrix.org" build \
                for matrix-org/olm

zinit id-as build"install install-doc-man" \
              atclone="asciidoctor -b manpage doc/*.adoc" \
              atpull"%atclone" \
                  for jonas/tig`

zinit id-as build for universal-ctags/ctags

How Has This Been Tested?

I've been using the ice with no problems for about of 8 months with no problems, for a variety of software, like the above examples.

Types of changes

Checklist:

psprint commented 11 months ago

@vladdoster @pschmitt ping

psprint commented 10 months ago

@vladdoster: ping. I've revived the patch and expect a merge

psprint commented 10 months ago

@vladdoster: Why this still has not been merged?

vladdoster commented 5 months ago

Closing in favor of PR feat: configure, make, cmake, and build ices (#613).