Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Expose Interpolate for external tools#74

Closed
ibuildthecloud wants to merge 1 commit intodocker-archive-public:masterfrom
ibuildthecloud:expose-interpolate
Closed

Expose Interpolate for external tools#74
ibuildthecloud wants to merge 1 commit intodocker-archive-public:masterfrom
ibuildthecloud:expose-interpolate

Conversation

@ibuildthecloud
Copy link
Copy Markdown
Contributor

It is quite useful to parse a text file in the same exact way that
compose does. One can imagine this being useful for GUI applications or
custom parsers.

@ibuildthecloud
Copy link
Copy Markdown
Contributor Author

I started integrating the variable interpolation into some tools and have found that it is much nicer to be able to be able to interpolate the file the same way that libcompose does internally.

@ibuildthecloud ibuildthecloud force-pushed the expose-interpolate branch 3 times, most recently from 50ab851 to 374e1e7 Compare October 8, 2015 05:14
It is quite useful to parse a text file in the same exact way that
compose does.  One can imagine this being useful for GUI applications or
custom parsers.

Signed-off-by: Darren Shepherd <darren@rancher.com>
@vdemeester
Copy link
Copy Markdown
Contributor

@ibuildthecloud Seems ok to me 👍

One question though : should or shouldn't it be in a separate package (like libcompose/interpolate or something like the pkg/… package in docker)

@ibuildthecloud
Copy link
Copy Markdown
Contributor Author

Hmm. Yes I could move it to libcompose/interpolate. I like that.

@vdemeester
Copy link
Copy Markdown
Contributor

@ibuildthecloud still working on it or you would like me to carry it ? 😉

@vdemeester
Copy link
Copy Markdown
Contributor

Carried by @joshwget in #181

vdemeester added a commit that referenced this pull request Mar 27, 2016
Carry #74 Expose Interpolate for external tools
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants