Skip to content

enabling runtime argument checking for free (part 1) #486

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
paolopas wants to merge 2 commits into from
Closed

enabling runtime argument checking for free (part 1) #486

paolopas wants to merge 2 commits into from

Conversation

@paolopas
Copy link
Contributor

@paolopas paolopas commented Dec 29, 2025 •
edited
Loading 

ECHO [ DEPENDS  a : b : c ] ;

actual behaviour

# b2

...found 1 target...

expected behaviour

# b2
jamfile.jam:1: in modules.load
*** argument error
* rule DEPENDS ( targets1 * : targets2 * )
* called with: ( a : b : c )
* extra argument c
(builtin):see definition of rule 'DEPENDS' being called
/home/pax/.local/share/b2/src/build/project.jam:586: in load-jamfile
/home/pax/.local/share/b2/src/build/project.jam:84: in load
/home/pax/.local/share/b2/src/build/project.jam:241: in project.find
/home/pax/.local/share/b2/src/build-system.jam:639: in module scope

the same goes for INCLUDES

@paolopas
argument description for DEPENDS/INCLUDES builtins
d4a576d 
now available, this when provided is used by the functions
function_bind_variables, function_run, argument_error allowing
the checking of arguments that otherwise cannot happen
@grafikrobot
Copy link
Member

grafikrobot commented Dec 29, 2025

@paolopas if you are rummaging around.. We could use help in rewriting the code to C++. :-)

@grafikrobot grafikrobot added the enhancement New feature or request label Dec 29, 2025
@grafikrobot grafikrobot moved this to 👀 In review in BFG Tasks Dec 29, 2025
@paolopas
argument description provided for builtin rules
f65f30a 
ALWAYS, LEAVES, NOCARE, NOTIME, NOTFILE, NOUPDATE,
TEMPORARY, ISFILE, FAIL_EXPECTED, RMOLD
@paolopas
Copy link
Contributor Author

paolopas commented Dec 29, 2025

oops, you were too quick, I also added support for the ALWAYS, LEAVES, NOCARE, NOTIME, NOTFILE, NOUPDATE, TEMPORARY, ISFILE, FAIL_EXPECTED, RMOLD rule arguments to this PR

@paolopas
Copy link
Contributor Author

paolopas commented Dec 29, 2025

@grafikrobot René I'll reply to you by email

@paolopas paolopas mentioned this pull request Dec 30, 2025
Closed
@paolopas paolopas mentioned this pull request Jan 7, 2026
Merged
@grafikrobot
Copy link
Member

grafikrobot commented Jan 7, 2026

Replaced by #492

@grafikrobot grafikrobot closed this Jan 7, 2026
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in BFG Tasks Jan 7, 2026
@paolopas paolopas deleted the builtins-1 branch January 8, 2026 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@grafikrobot grafikrobot grafikrobot approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

BFG Tasks
Status: ✅ Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@paolopas @grafikrobot