[pacman-dev] [PATCH 2/2] meson: add toplevel target to create the website

[Dave Reisner]

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 at 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...