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.

Cleanup: Simplify logic in fake_validate_parameter_paths().

Bugfix: Short console arguments should not match when a single invalid exists.

While convenient, "fake -help" should not print help.
This is because there is no short command starting with the letters "e" or "l".

This behavior of succeeded is confusing and misleading.

Redesign the logic to better handle this.

Improve the unit tests to catch more cases, including this one.

Cleanup: Use the initializer define for initializing f_console_id_t.

Cleanup: Add missing enumeration end comment.

Cleanup: Simplify logic at wrap up in f_console_parameter_process().

Cleanup: Console style and remove extra integer definition.

The declaration of the "f_array_length_t i = 0" is not necessary.

Cleanup: Remove unused global static constant strings.

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.

Bugfix: Incorrect stream used when printing output in verbose directory recursion.

I somehow ended up with the input stream being written to.
This is clearly wrong.

How did I not notice this one?

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

Bugfix: The stream is locked again rather than unlocked after being locked.

Cleanup: Use 0 instead of F_false in firewall initialization.

Mathematically, the F_false and 0 are synonymous.
Semantically, the F_false represents a boolean and 0 represents a digit.

Using F_false here has incorrect semantic meaning.

Update: Next micro version (0.6.4).

Update: The stand alone programs configuration.

Refresh the disable define configuration.
Fix problems in the settings files.
The stand alone fake program now has threading enabled (It was previously broken).

Update: The disable header files.

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

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.

The firewall incorrectly has f_account as a dependency.

Bugfix: The f_int_128_t is using incorrect size.

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

Bugfix: Fix wrapping define macros for several functions across different projects.

Cleanup: remove trailing 's' from _di_f_print_format_flag_.

Cleanup: Remove extra underscore in _di_f_color_context_t_.

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

Update: Add stand alone fss_status_code build.

Update: Add stand alone status_code build.

Cleanup: Remove unnecessary dependencies from fss_status_code.

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: Add missing 'r' in 'other'.

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: Minor fixes regarding structure and output relating to help.

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

Update: Add man page documentation for status_code.

Cleanup: Use correct year and remove extra line.

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.

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 25671e39d0a275b4a14b1db6612ae46d497c38c7 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: 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 4dfc152aad390be408cb9a20578b337b2a13d993 didn't include checking and updating the unit tests.

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

Cleanup: Replace 'decision' with 'choice' for consistency reasons.

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 'main' is NULL.

Cleanup: Remove stale and no longer valid comment about interrupts and signals.

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.

Update: Clarify wording in specifications.

Update: FSS Status Code should have a fakefile.

Update: Next micro version (0.6.3).

Cleanup: Remove improper white space from build fake files.

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.

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.

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.

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.

Update: Add not implemented clarifications for 0.6.x versions.