Security: NULL pointer dereference in writer due to pipe function result handling.

The GCC -fanalyzer parameter helped me discover this one.

The status_pipe is being read and processed.
There is a case where the status_pipe is being set but it is not being reset after handling.
In a later loop the pipe does not get read but the previously set state is used bringing the code into a bad state.
Then the loop doesn't do the block buffer used check and this results in the eventual NULL dereference.

Update: Add fanalyzer build mode.

I keep forgetting to run the GCC fanalyzer sanity checks to help catch problems before making a release.
Add a formal mode in all of the build settings to make this processor easier and therefore easier to remember.

Add missing -Wall to some of the files for the test mode.

Update: Use pre-existing cache rather than using a new variable.

This potentially reduces reallocations.

Bugfix: Further Fakefile processing with iki replacement space problems.

This is a follow up to commit 3713a5f0bd90d32f564afaca29f01d9ccf329cfb.

The other IKI variables still need the new lines when the context does not.

Detect when certain IKI variables expand into nothing and in these cases do not append.
When performing last line, only append if it is not by itself.

Cleanup: Reorder logic in FSS Basic Read function.

Bugfix: Fakefile processing with color context is producing extra spaces when unquoted.

Do not generate a new argument when calling fake_make_operate_expand_context().
The color context should combine with existing arguments.
This does not handle the last context argument and more work is likely needed to handle that one.

Remove the not quoted check that always generates a new argument.

Bugfix: FSS Write programs don't correctly handle multi-Content per Object situations.

The FSS Extended should support one or more Content per Object.
All others support the same number of Objects as Content (not referring to nested Content).

The error and error message when this is and is not correctly being handled.

The correct detection needs to count based on sub locations rather than locations.

Remove now pointless and misleading checks.

Progress: Continue program related work and make f_console related changes.

Work on fake and then focus on the relating console changes.

I imported the recent bugfix regarding console arguments (ccd15d4e64848f29d36823ca92ee639057f544b7).
I opted to just copy and paste and then re-convert to the latest practice.
Look out for regressions, but this is development anyway.

I've added it to the console function.
As far as how the main programs declaring and using f_state_t to pass to the console program, I'm not sure how I want to handle f_state.

Progress: Continue program related work, focusing primarily on fake and console parameter handling.

I realized that the new callback console parameter needs to be implemented.
I reviewed the drafted design and changed it accordingly.
I've been able to reduce the need to have a callback for each parameter and have a single callback for the set of parameters.

The documentation for the console parameter callback change is incomplete for this and my next progress should complete this documentation.

I updated several programs regarding the console changes and noticed a small number of bugs in the UTF8 program.
The first line is now being printed, conditionally, but future work may be needed.

Progress: Continue program related work.

Progress: Continue program related work.

This focuses on getting the Featureless Make program to compile.
The settings and functionality is not yet updated so the program is not expected to work yet.

Update: Add locale environment variable support, install, and help support in fakefiles.

Pass along the common locale-specific environment variables along when using a fakefile.

Add an "install" operation that is essentially a wrapper to the "install.sh" script.

Add a "help" operation to provide some documentation.

These are provided as a convenience and as a functional proof of concept.

Progress: Continue program related work.

Focused on Featureless Make.
Other changes made as convenient.

Update: The directory documentation comments.

Update: The directory documentation comments.

Update: Use _di_f_type_d_ instead of _di_f_type_input_output_d_.

Update: Add new standard global type, "_g".

Update: The disable header files.

Bugfix: Fix some of the outdated unit tests.

The unit tests are out of date.
Fix a few of the obvious problems.
There will be more problems to solve that are not solved in this commit.

Bugfix: Several problems with the wrapping define macros, unused private functions, and non-existent functions.

The stand alone builds revealed several problems with the wrapping define macros.
The ones that I noticed or were blocking me from compiling have been fixed.
It will be no surprise if there are still more such problems hiding in the code somewhere.

Remove stale and unused private functions.

There is are no f_file_clone_at() and f_file_copy_at() functions.
Remove references and related wrapping define macros.

Bugfix: The f_int_128_t is using incorrect size and change f_int_128_t to f_int128_t.

The wrong wrapping define macros are being used.
- Use __SIZEOF_INT128__ instead of _di_f_int_128_t_.
- Use #ifdef instead of #ifndef.

Use f_int128_t to more closely match core types like int64_t.

Cleanup: Add new lines to dependencies file in level 3 projects.

Bugfox: The type macros in type.h are missing 'f_' or have spurious '_type' in the name.

Bugfix: Incorrect define macros for private functions in f_file.

Bugfix: Incorrect macro definition _di_pthread_support_ vs _di_thread_support_.

The _di_pthread_support_ is used in the project but all of the programs have _di_thread_support_.
The project name is f_thread and not f_pthread so favor _di_thread_support_ over _di_pthread_support_.

Cleanup: Fix license confusion.

The documentation should be under the cc-by-sa-4.0 license.
However, the specification must be under the open-standard-license-1.0 license.

This file is documentation on the time specification.
If the documentation is under another license, the standard it describes must still be followed according to the open-standard-license-1.0 license.
The documentation would have to communicate that the standard it is describing is still under the open-standard-license-1.0 license even if the documentation itself is under a different license.
This creates a confusing and misleading situation.
Clarify the situation by setting the license to open-standard-license-1.0.

Setting the license explicitly to open-standard-license-1.0 should hopefully address any misunderstandings.

Cleanup: Parameters in byte_dump man page should be bold.

Update: Documentation and build settings, adding remaining man pages for programs.

This includes any updates to the existing program man pages already added.

Update: Use more proper copyright communication.

The original copyright file was added to the git repository in 2011.
The svn repository has the actual original initial date, 2007.

Cleanup: Minor IKI formatting improvements.

Cleanup: Apply IKI formatting to controller specifications and documentation.

Update: Add man page documentation for iki_read.

Update: Add man page documentation for iki_write.

Update: Add man page documentation for status_code.

More work is needed to only install the man pages for each specific program.

Cleanup: Use correct year, remove extra line, and fix version.

Update: Improve IKI Read help documentation.

Update: Improve IKI Write help documentation.

Update: Add more detailed documentation to fss_status_code and status_code.

Bugfix: Incorrectly string length causes inability to match string in fss_status_code.

Update: Restructure settings now the install.sh is installing them.

These files are moved into a sub-directory.

Update: Improve logic and design of install.sh script.

The destination directories should be created if they do not already exist.
Fail when the destination exists but is not a directory rather than fail if that directory does not exist.

Fix grammar in 'Failed to create work directories'.

The includes should be created only when work is not an empty string.
The recently added documentation needs to be created as well.
The settings is not created and needs to be created just like with documentation.

Ensure the existence of these directories allows the copy and paste to work correctly.
Ensure the normal directories get created in addition to the work directories.

I originally designed this to not have any opinions for the settings.
I've decided to relax that and apply the default settings of the 'etc/' directory.
This directory is now installed to.
If say the 'etc/' is actually needing to be the 'share/' directory, custom paths can be altered as well.

Make sure the documentation and settings can be disabled or have their destination paths altered.

Update: Add man pages for utf8 program.

Feature: Add missing documentation handling functionality to Featureless Make.

This is an oversight and should have been implemented before the first stable release.
This feature should be a core feature.

Documentation is important.
Make sure the handling of documentation is available for use.

This adds "build_sources_documentation" to the build settings to help facilitate documentation installation.
The documentation that is copied is expected to be under 'data/documentation/'.

The 'documentation' and 'documents' terminology overlaps to some degree.
The 'documentation' is a more specific thing and may not always be a 'document'.
The existing behavior around 'documents' is left unchanged.

The skeleton operation now creates the 'data/documentation/' directory.

The bootstrap.sh script now supports enabling and disabling documentation via the more common '--enable-doc' and '--disable-doc' parameters.

Cleanup: Fake program has incorrectly named variables.

The variables with 'build_sources_settings' should not have an 's' on the end.
The variables with 'build_process_post_s' and 'build_process_pre_s' should not have an '_s' on the end except for when it is a string.

Update: Add explicit note about copyrights belonging to Kevin Day and the year 2023.

Regression: A  bad refactor caused the "file" and "output" got changed into "print.to" in the utf8 program.

Cleanup: Fix grammar in utf8 program help.

Bugfix: The mode clang is missing from the UTF8 stand alone build settings.

Bugfix: Support for passing a fakefile Section incorrectly fails.

The commit 6c05c66f50719c9edaa36dea75669e03cd558ee4 added a loop around the code.
I forget to handle the return status and break out of the loop appropriately.

Move the variable declaration outside of the loop to avoid re-declaring it inside.

An error or a child signal should cause the loop to terminate.

When the string comparison is matched, do not continue processing the inner loop.

Cleanup: Use correct version in install.sh script.

I probably got lazy and copied then install.sh script from the 0.7 development branch and then forgot to update the version.

Feature: Add support for installing documentation to the install.sh script.

Cleanup: Use braces around variable names.

Do this by practice now to better avoid ZSH and other potential compatibility problems.

Bugfix: Install script still install static program sources despite being disabled.

The if condition nested is only wrapping the print block.
Move the conditional logic to the proper wrapping block.

Feature: Support passing a fakefile Section name to the main program.

This is intuitive and should have already been supported in the project to better transition from GNU Make into Featureless Make.

The unknown parameter error is to no longer be thrown.

Instead, when the make process in being followed, check to see if any Sections exist in the fakefile matching the main program Section parameters.
Fail when this happens.
If the Section exists in the fakefile then that Section is operated on.

This, for example, now allows for doing something like "fake install" in the same way "make install" works.

The "install" Section must exist in the fakefile for this to work.

Cleanup: Add missing space in fake --help output.

Update: De-couple the build settings data file from the install.sh script.

The design is originally focused around the data/build/settings file.

This file is simple enough and easy enough to use that I decided to use it beyond its intended design.

I noticed that the data/build/settings isn't required anymore.
Removing this allows me to better use in other projects, such as those that have only a fakefile or have a different directory location for the build settings data.

The install.sh script is even simpler now.

Update: The f_file_mode_from_string() replace parameter should be optional.

This change does not break ABI.
This change does not break API, except for parameter error when replace is NULL.

Feature: Add f_file_is_stat() to perform f_file_is() using a pre-populated file statistics structure.

Update: Be more thorough in the f_file unit test.

The "*_returns_false" test is doing a more complete check but the "*_returns_true" is not.
Redesign the "*_returns_true" test to have the same structure as the "*_returns_false" test.

Bugfix: The f_signal unit tests after significant change.

The bugfix from commit 3a5a4e73f16b0285e9cfb436481835e192896ef8 didn't include checking and updating the unit tests.

Update: Use an enumeration rather than a define macro for the conversion flags.

Update: Code documentation is missing the function fl_conversion_dynamic_partial_to_unsigned_detect().

Bugfix: The signal is not properly closing.

The signal id is a file descriptor.
File descriptors are unset when at -1 rather than 0.

The not zero check is therefore invalid.
Replace that check with a check against negative one.

Security: Check if either 'main' or 'setting' is NULL.

Name the void pointer 'setting' as 'void_setting' for consistency purposes.
Note that this should not be named 'void_setting' in the header files.

Update: Do not treat interrupt not being listened to as an error.

When the interrupt is received and the interrupt is not in the designated set then an error is being returned.
Change this behavior to return a non-error status.

Security: Check if setting is NULL.

Cleanup: Use 'choice' instead of 'decision'.

This became inconsistent at some point in time.
Use 'choice' simply because it is shorter.

Update: Use f_color_mode_not_e rather than f_color_mode_color_not_e.

The second 'color' in f_color_mode_color_not_e is redundant and inconsistent with the other parameters.
Remove it.

Update: Fix license in documentation.

The time.txt specification uses the open-standard-license-1.0.
When I separated the time.txt and formally wrote the specification for the standard, I forgot to set the documentation to a documentation license.
The documentation licese is supposed to be cc-by-sa-4.0.

Bugfix: Not defaulting to top-level fakefile when data/build/ does not exist.

This is happening because the `data/build` directory check is happening independent of the defaulting to the local fakefile step.

Add a check that removes the requirement only when there is no explicit fakefile parameter specified and a fakefile at the top-level exists.

Feature: Add support for a prepending a prefix to build directory names.

This should make it easier to build packages for releases with the packages named for "monolithic-", "level_0-", "stand_alone-", etc...

Bugfix: The "-mode" functionality is incomplete and incorrect in the bootstrap.sh script.

The bootstrap_load_settings_has() method is removed for being incorrect.

Add missing Objects, such as build_indexer, build_indexer_arguments, build_name, search_exclusive, and search_static, to the "has-" detection data array.
Add additional detection data array for the "-mode" variants of the "has-" data array keys.

Stop checking if Content is an empty string when doing an Object existence check.
Just check to see if the Object exist, with or without Content.

Override the parameters when a "-mode" is provided for single-value parameters, even if the Content does not exist.
This should properly fail for when the Object is overridden with a mode that has no Content and is required.

For example, take the following:
  build_compiler gcc
  build_compiler-clang

In that above case clang is a mode that is overriding but has no Content.
The build_compiler is required.
This should fail because clang mode replaces the single-valued build_compiler with no Content.
This is a proper failure.

Cleanup: Fix comments that have incorrect notes.

Cleanup: Fix comments that have incorrect notes.

Progress: Prototype the callback paameter for console parameters.

There needs to be a way to have conditional parameters and other relationships.
Provide an optional callback to help facilitate this without adding any additional complicated structures (other than the callback itself).

I need to figure out how I want the arguments to the callback to be defined.
This stubs out the design.
The parameters to the callback are subject to change once I determine what they should be.

Update: Documentation and specifications.

All of the documentation and specifications should now be IKI based.

I have finally decided on how I want abbreviations handled.
This removes the "abbreviation-" and "abbreviation:" from use.
The new behavior has the abbreviation followed by the expansion, such as FLL:"Featureless Linux Library".
have the block IKI structure identical in pattern to the script IKI structure.

Use "italic" instead of "emphasis" for such IKI vocabulary.

Update: Add formal specification for Time and EpochTime and use IKI in the files.

Cleanup: Bring a copy of time.txt from Controller to the core project.

New functionality needs to be added to the package.sh script to conditionally add documentation and specifications to any given package.
Once this is done, then the now redundant time.txt can be removed from the Controller and possibly Control projects.

There also needs to be a formally specification file for time.txt and not just documentation.

Cleanup: Re-word sentences in time.txt to be less likely to be mis-interpreted.

Cleanup: Be more specific about quotes in FSS and add missing unicode value.

Bugfix: The IKI variables that expand reserved words should expand into multiple parameters.

Reserved words, such as the ones regarding the --mode parameter, have multiple values then these should themselves expand into multiple values.
Wrapping quotes can be added to prevent this behavior.

There may be previous issues that intentionally added this (and then possibly as a bugfix).
The previous behavior of expanding multi-valued reserved words into a single parameter value when not using quotes is now considered a bug.

Update: Slight changes to design around the build settings arguments.

Improve the argument handling when the fakefile contains a build settings with arguments.

This is done in observance of a bug where passed modes are combined into a single parameter when modes are passed via an IKI replacement.
A follow up bugfix that may involve some refactoring or functionality reinterpretation is needed.

This uses an explicit array when needed rather than an inexplicit pointer.

This repeats the parsing when printing the output so that information output is not misleading.

Cleanup: Fix problems with comments.

Bugfix: The fakefile needs to have the supported modes.

Security: Ensure pointer exists before calling process function callbacks.

In cases where these are not defined, just do nothing.

Progress: Begin working on Featureless Make.

These are the initial changes I made while looking at what I am going to need to do.

Update: Improve some of the Featureless Make documentation and specifications.

There will likely be another pass at this.

Update: Build and other documentation, fixing or adding IKI support.

Update: Add lossy allocation documentation.

Cleanup: The main program header files should not normally be installed.

Update: Add main.h to every program.

This should not be installed as it is for the program and not the library.
This just provides documentation for most cases.

Bugfix: _GNU_SOURCE should be wrapped in an ifndef.

Update: Out of date build documentation.

The "See: ..." files should now end in ".txt".

The directory structure for both the status project and the work directory have changed since this documentation was last modified.

Update documentation regarding running Featureless Make with no arguments.

Remove redundant "Alternatively..." documentation that is already described elsewhere.

Bugfix: The bootstrap script needs to support settings-specific built files.

Bugfix: The bootsrap script needs to treat -s/--settings as a possible path.

Update: Add fakefile for the FLL monolithic and level builds.

Bugfix: Incorrect directory path for header file.

Cleanup: Rename files in status_code project.

Feature: Add 'import' Object to fakefile and settings specifications.

This is important for building complex project structures and reducing the code redundancy.
To keep the logic and design simple, recursion is not allowed.

One can now create a base settings file that is imported by other settings files.
The same is true for fakefile files.

Cleanup: Fix styling, spelling, and wording mistakes in the specifications and documentation.