Skip console confirmation of "upload"

[yegor256]

I execute l3build upload in a script, which is run by a machine (not a human being). Now, I have to configure it like this:

echo 'y' | l3build upload --halt-on-error

Would be much more convenient to have an option for this:

l3build upload --halt-on-error --yes

[davidcarlisle]

see l3build-upload.lua you can set ctanupload to true and it won't ask

[yegor256]

@davidcarlisle I should add l3build-upload.lua to my repository? Now I only have build.lua.

[yegor256]

@davidcarlisle this code inside build.lua has no effect:

module = "iexec"
ctanupload = true
uploadconfig = {
  pkg = "iexec",
  ctanupload = true
  ...
}

[davidcarlisle]

line 2 should be enough as far as I can see, l3build does

local ctanupload = ctanupload or "ask"

so should be true if you set it to true, then the dialog Do you want to upload to CTAN? [y/n] is guarded by

if (ctanupload ~=nil and ctanupload ~=false and ctanupload ~= true) then

so it should be skipped.

Can you make a small test file (while testing you can use debug option which makes it use https://httpbin.org/post rather than ctan so you can test the form post without bothering the ctan maintainers.

[yegor256]

@davidcarlisle this is my config: https://github.com/yegor256/iexec/blob/master/build.lua This is the result of the build (scroll to the bottom): yegor256/iexec#12

[yegor256]

@davidcarlisle maybe I'm using some old version of l3build? I have this:

$ l3build --version

l3build: A testing and building system for LaTeX

Release 2022-04-19
Copyright (C) 2014-2021 The LaTeX Project

[davidcarlisle]

@yegor256 sorry you are right

      local ctanupload = ctanupload or "ask"

in l3build-upload.lua should not be there, this file is read before your setting so it always sets to ask

instead the end of l3build-variables.lua should be

-- Upload settings
curlexe  = curlexe  or "curl"
uploadconfig = uploadconfig or {}
ctanupload = ctanupload or "ask"

with ctanupload line being added.

It then gets set to ask only if not set as that file is read after build.lua

@josephwright does that look right?

Incidentally I had to change

if string.match(file, "%.dtx$") || string.match(file, "%.lua$") then

to

if string.match(file, "%.dtx$") or string.match(file, "%.lua$") then

in your build.lua

I'd also note that we wouldn't normally recommend automated uploads with no confirmation, ctan is not automated so every upload requires human volunteer time to handle the upload and respond, you do not normally want to do that on every commit for example.

[josephwright]

@davidcarlisle Yes that looks right: I'll adjust

[muzimuzhi]

I think this issue has been resolved by

• 223a4ed (Add ctanupload to l3build-variables, 2022-10-02) and
• 8dd0258 (ctanupload set in l3build-variables.lua now, 2022-10-03)

combined.

Update: Oops, I was wrong. 8dd0258 introduced a new issue, see #264.