On Sun, Jan 13, 2019 at 10:38:30PM -0500, Eli Schwartz wrote:
On 1/13/19 10:18 PM, Allan McRae wrote:
On 13/1/19 11:17 pm, Dave Reisner wrote:
On Sat, Jan 12, 2019 at 09:15:01PM -0500, Eli Schwartz wrote:
Signed-off-by: Eli Schwartz <eschwartz@archlinux.org> ---
This is really just an alias, right? To match what autotools does? Does anyone actually care about this? Might it be better to just rename the existing target to 'website' instead?
That seems the least redundant approach.
The existing target is specifically a file target, it's a command to create output. It only exists as:
ninja doc/website.tar.gz
The website target is a toplevel pseudo-target, you can run it directly from the build tree and it will depend on the tarball being up to date rather than always rerunning bsdtar. I don't like the idea of turning the whole thing into one always-rebuilding pseudo-target.
Why bother with this at all, then? Autotools offers website and website.tar.gz, but both are in doc/. This target also is called probably once per release. Aliasing it to some shorter (and sub-par) seems not so useful... I think we should just skip this...