BUDA: support multithread apps

[davidpanderson]

docker wrapper

• pass --cpus arg to Docker/Podman if app uses >1 CPU
• Win: output of stats command can start with warning line;
  skip it to parse CPU time correctly
• change version to 7
• Win: add \n to command output strings so they display correctly

multi_thread (test app)

• add --standalone option so you can run it as Docker app

web (job submission)

• allow apps with no input or output files

plan class XML spec:

• add <nthreads_cmdline> to mt class

[Copilot]

Pull Request Overview

This PR bumps the Docker wrapper version to 7 across build/config files, adds a new nthreads_cmdline XML setting, and refactors several sample applications and utilities to use new constants and improve robustness.

• Bump Docker wrapper release to 7 in project files and scripts
• Add <nthreads_cmdline> setting to plan_class XML
• Refactor multi-thread and Docker wrapper samples for clearer constants, improved parsing, and STANDALONE support

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
win_build/docker_wrapper.vcxproj	Updated <TargetVersion> from 6 to 7
version.h	Bumped DOCKERWRAPPER_RELEASE to 7, removed WSL_WRAPPER_RELEASE
sched/plan_class_spec.xml.sample	Added <nthreads_cmdline> element
samples/multi_thread/multi_thread.cpp	Replaced TOTAL_UNITS with UNITS_PER_THREAD, added STANDALONE guard
samples/multi_thread/Makefile	Renamed PROGS to PROG, updated build target
samples/docker_wrapper/docker_wrapper.cpp	Added --cpus flag, robust stats parsing, fixed fprintf call
lib/str_util.cpp	Modified split to append delimiter
html/user/buda.php	Reordered handling of output_file_names
configure.ac	Bumped DOCKERWRAPPER_RELEASE to 7

Comments suppressed due to low confidence (3)

sched/plan_class_spec.xml.sample:7

• A new <nthreads_cmdline> element was added—please update the plan_class_spec documentation to describe its purpose and valid values.

        <nthreads_cmdline>      1           </nthreads_cmdline>

version.h:32

• The WSL_WRAPPER_RELEASE macro was removed but may still be used elsewhere; please reintroduce #define WSL_WRAPPER_RELEASE 4 (or appropriate version) before bumping DOCKERWRAPPER_RELEASE.

// dockerwrapper version number

lib/str_util.cpp:842

• Appending the delimiter to each split item changes the expected behavior of split; most consumers expect the delimiter to be removed. Consider pushing item alone.

        result.push_back(item+delim);

[Copilot]

The distclean rule still references $(PROGS) which is no longer defined; update it to use $(PROG) so the generated binary is cleaned up correctly.

Suggested change

	/bin/rm -f $(PROGS) $(addsuffix .exe, $(PROGS)) *.o libstdc++.a
	/bin/rm -f $(PROG) $(addsuffix .exe, $(PROG)) *.o libstdc++.a

[Copilot]

The units_per_thread variable is no longer used after switching to the UNITS_PER_THREAD constant; consider removing this declaration to avoid confusion.

Suggested change

int units_per_thread;

[Copilot]

[nitpick] Iterating by value causes unnecessary string copies; prefer for (const string& line : out) to avoid allocations.

Suggested change

	for (string line: out) {
	for (const string& line : out) {