]> Kevux Git Server - fll/log
fll
9 months agoCleanup: Remove out of place semi-colon from documentation comments.
Kevin Day [Fri, 23 Feb 2024 03:37:54 +0000 (21:37 -0600)]
Cleanup: Remove out of place semi-colon from documentation comments.

9 months agoCleanup: Fix wording in documentation comment for 'compression'.
Kevin Day [Fri, 23 Feb 2024 01:57:10 +0000 (19:57 -0600)]
Cleanup: Fix wording in documentation comment for 'compression'.

9 months agoCleanup: Fix grammar in documentation comment, replacing process with processes.
Kevin Day [Thu, 22 Feb 2024 05:05:07 +0000 (23:05 -0600)]
Cleanup: Fix grammar in documentation comment, replacing process with processes.

9 months agoCleanup: Use F_true and F_False instead of TRUE and FALSE in the documentation comments.
Kevin Day [Thu, 22 Feb 2024 03:50:12 +0000 (21:50 -0600)]
Cleanup: Use F_true and F_False instead of TRUE and FALSE in the documentation comments.

9 months agoCleanup: Function documentation comments, removing or fixing invalid comments.
Kevin Day [Wed, 21 Feb 2024 03:37:38 +0000 (21:37 -0600)]
Cleanup: Function documentation comments, removing or fixing invalid comments.

The Doxygen wizard reported warnings that exposed these problems.

9 months agoUpdate: Back port the ability to explicitly skip a program in the bootstrap example...
Kevin Day [Wed, 21 Feb 2024 02:02:38 +0000 (20:02 -0600)]
Update: Back port the ability to explicitly skip a program in the bootstrap example script.

The controller program is not yet migrated to compile under 0.7.x.
This adds the ability to the bootstrap example script to allow for simply skipping packages.

9 months agoUpdate: Back port settings configurations, such as gcc and flags.
Kevin Day [Tue, 20 Feb 2024 13:25:26 +0000 (07:25 -0600)]
Update: Back port settings configurations, such as gcc and flags.

Back port the gcc mode.
Back port the -Wno-missing-braces flag.

Make sure the flags are used more consistently within the build settings.

9 months agoUpdate: Backport fixes for bootstrap-example.sh, package.sh, and test.sh scripts.
Kevin Day [Tue, 20 Feb 2024 06:17:59 +0000 (00:17 -0600)]
Update: Backport fixes for bootstrap-example.sh, package.sh, and test.sh scripts.

Add support for "-m gcc".
- A follow up backport commit for every build settings file and fakefile is needed after this commit.

Fix issue with test_operate_build_project not getting the test_thread_individual variable.

See commits:
a850c4fc237e346cecc70bf3ed87b958299ceca2
ea1c3d46fecaa3e267bb9900e509a43026a85648
ba083b0d08191cb03f3531ef64ae82c881318f04

9 months agoUpdate: Backport improved return code failure reporting to firewall from 0.7 development.
Kevin Day [Sun, 18 Feb 2024 04:11:20 +0000 (22:11 -0600)]
Update: Backport improved return code failure reporting to firewall from 0.7 development.

The return code may be returned as non-zero for error returned by iptables.
This error is now being reported.
This error does not prevent the firewall from continuing.

9 months agoBugfix: De-allocation call is missing in firewall program when "firewall stop" is...
Kevin Day [Sun, 18 Feb 2024 03:48:32 +0000 (21:48 -0600)]
Bugfix: De-allocation call is missing in firewall program when "firewall stop" is used.

Fix a spot where the de-allocation is not performed.

9 months agoBugfix: The debug mode is not working in firewall.
Kevin Day [Sat, 17 Feb 2024 16:26:54 +0000 (10:26 -0600)]
Bugfix: The debug mode is not working in firewall.

The debug verbositity mode is at the 5th position in the array and is not being used because the used length is set to 4 instead of 5.

9 months agoBugfix: Firewall length check from range is not calculating 0 correctly.
Kevin Day [Thu, 15 Feb 2024 04:47:49 +0000 (22:47 -0600)]
Bugfix: Firewall length check from range is not calculating 0 correctly.

When the range.start is greater than the range.stop, then the length is 0.
Rather than checking for this, this just subtracts range.start from range.stop then adds 1.
This results in the case of say (0 - 1) + 1, which may not be 0 due to overflow behaviors.
Play it safe and explicitly test for this rather than hoping that the overflow operates ideally.

9 months agoCleanup: Replace two condition checks with one.
Kevin Day [Thu, 15 Feb 2024 03:05:43 +0000 (21:05 -0600)]
Cleanup: Replace two condition checks with one.

If less than or equal to zero or greater than zero is the condition, then the simpler condition would be to just check not equal to one.

9 months agoBugfix: Firewall program if condition is setting an error flag rather than checking...
Kevin Day [Tue, 13 Feb 2024 00:56:47 +0000 (18:56 -0600)]
Bugfix: Firewall program if condition is setting an error flag rather than checking the value.

9 months agoBugfix: Firewall program is not printing error when fll_fss_extended_read() fails.
Kevin Day [Sun, 11 Feb 2024 03:32:10 +0000 (21:32 -0600)]
Bugfix: Firewall program is not printing error when fll_fss_extended_read() fails.

9 months agoBugfix: Error message is saying "lock" when it should say "stop".
Kevin Day [Sun, 11 Feb 2024 03:04:44 +0000 (21:04 -0600)]
Bugfix: Error message is saying "lock" when it should say "stop".

9 months agoUpdate: Make sure stand alone builds have consistent buid mode settings.
Kevin Day [Fri, 2 Feb 2024 04:49:08 +0000 (22:49 -0600)]
Update: Make sure stand alone builds have consistent buid mode settings.

Make sure "-pthread" is present via the "thread" mode.

Add any missing flag modes such as "coverage" and "test".

9 months agoUpdate: Add hopefully more secure compiler flags by default.
Kevin Day [Thu, 1 Feb 2024 05:37:12 +0000 (23:37 -0600)]
Update: Add hopefully more secure compiler flags by default.

Use the "-Wl" for specifying the linker flags.
- Make sure "now", "relro", and "nodlopen" are set.

Add FORTIFY_SOURCE set to 3.

Add stack-clash-protection and strict-flex-arrays set to 3.

This project is designed around NULL checks.
- Make sure the no-delete-null-pointer-checks flag is set to prevent the compiler from removing these security/integrity checks.

Use stack-protector-strong rather than stack-protector for test flags.
- Future versions may enable stack-protector-strong by default for regular compiling.

9 months agoUpdate: Make the -m thread option in the example bootstrap script more easily customi...
Kevin Day [Thu, 1 Feb 2024 05:23:37 +0000 (23:23 -0600)]
Update: Make the -m thread option in the example bootstrap script more easily customizable.

9 months agoBugfix: FSS Basic List and FSS Extended List print comment at the start of the Content.
Kevin Day [Wed, 31 Jan 2024 05:41:52 +0000 (23:41 -0600)]
Bugfix: FSS Basic List and FSS Extended List print comment at the start of the Content.

When the Content exists at the start of the Content, the comment character ('#') is printed when it should not be printed.

This is a bug where the code initializes the newline_last at the range.start.
The code logic then always expects the newline_last to represent an actual new line.
This is not necessarily the case for when newline_last is pointing to the initial range.start position.

Add a check when processing a comment to ensure that the newline_last is in fact a new line.

9 months agoCleanup: Bad documentation comment syntax in fll_fss functions.
Kevin Day [Sun, 28 Jan 2024 06:45:54 +0000 (00:45 -0600)]
Cleanup: Bad documentation comment syntax in fll_fss functions.

10 months agoBugfix: FSS Extended read fails to handle non-terminated quote as per specification.
Kevin Day [Thu, 25 Jan 2024 04:50:30 +0000 (22:50 -0600)]
Bugfix: FSS Extended read fails to handle non-terminated quote as per specification.

This is a follow up to the problems discovered while writing unit tests (from the 0.7 development branch) and referenced in this commit: 70cbbe34121dc5679961df711e70724f13104489.

Given the line:
  a " b c d.

The following Content should now be returned:
  [0] = "
  [1] = b
  [2] = c
  [3] = d.

This adds a new FSS state flag 'f_fss_state_quote_not_e' to give the caller the ability to manually designate that the quotes are being disabled.
Currently only the FSS Extended utilizes this flag.

Refactor the private function private_fl_fss_basic_read() into private_fl_fss_basic_or_extended_read() to make it more explicitly clear that it provides functionality fo both FSS Basic and FSS Extended.

This changes to the code such that when an unterminated quote is detected then the calling function will set the disable quote flag and then call the function again.

10 months agoBugfix: Incorrect stop position is calculated when FSS content ends at the start...
Kevin Day [Wed, 24 Jan 2024 05:53:34 +0000 (23:53 -0600)]
Bugfix: Incorrect stop position is calculated when FSS content ends at the start position on FSS read operations.

When the start position is say, 0, and the determined stop position ends up being 0, then an incorrect stop range is calculated.
This happens because the stop position is subtracting one from the current position.

Add checks to ensure that the stop position is never subtracted beyond the initial start position.

The initial start position is saved at the beginning of each affected FSS read function.

This now potentially returns a start range before the stop range.
The FSS read programs should also need to be updated following this commit to handle these cases.

10 months agoCleanup: Use correct project name in fll_path documentation comments.
Kevin Day [Fri, 19 Jan 2024 03:12:37 +0000 (21:12 -0600)]
Cleanup: Use correct project name in fll_path documentation comments.

10 months agoCleanup: Invalid environment define "CMOCKA_TEST_ABORT" in testfiles.
Kevin Day [Fri, 19 Jan 2024 03:07:32 +0000 (21:07 -0600)]
Cleanup: Invalid environment define "CMOCKA_TEST_ABORT" in testfiles.

This invalid configuration doesn't break anything unless someone uncomments it.
Then, the expected functionality does not happen.
The fix is easy, just add "define " at the start of that line to the right of the comment character.

10 months agoBugfix: Functions private_fl_fss_basic_read() and private_fl_fss_basic_write() are...
Kevin Day [Thu, 18 Jan 2024 02:47:23 +0000 (20:47 -0600)]
Bugfix: Functions private_fl_fss_basic_read() and private_fl_fss_basic_write() are missing "const".

The function implementations for private_fl_fss_basic_read() and private_fl_fss_basic_write() are missing the "const" type attribute in some parameters.
The declaration of these has the "const" type attribute as expected.

This inconsistency is considered a bug.

10 months agoRefactor: The backtick into the grave, for the specifications.
Kevin Day [Tue, 16 Jan 2024 15:55:47 +0000 (09:55 -0600)]
Refactor: The backtick into the grave, for the specifications.

This is a terminology change in the specifications and the code (both API or ABI) does not need to be changed.
Changing the code would be ideal, but this would break API and likely ABI.

The 0.6 version series will continue to use the word "backtick" in the code itself for this reason.

10 months agoUpdate: Add additional parenthesis around status in F_status check macros.
Kevin Day [Sun, 14 Jan 2024 00:01:29 +0000 (18:01 -0600)]
Update: Add additional parenthesis around status in F_status check macros.

This is an extra safety check just in case.

10 months agoUpdate: Optimize bitwise logic regarding removing bits.
Kevin Day [Wed, 3 Jan 2024 23:22:17 +0000 (17:22 -0600)]
Update: Optimize bitwise logic regarding removing bits.

I intended to eventually do this and I finally got around to it.
Change the logic to use "x &= ~y" rather than "x -= x & y" or "if (x | y) x -= y".

In the case of "x -= x & y", the resulting object code is identical in my tests.
However, the code is simpler to write with the "x &= ~y".

The "if (x | y) x -= y" results in a lot more commands in the object.

10 months agoUpdate: The FSS-000E specification needs to say "one required object" and not "two...
Kevin Day [Mon, 1 Jan 2024 16:38:12 +0000 (10:38 -0600)]
Update: The FSS-000E specification needs to say "one required object" and not "two required objects".

This is an oversight from the commit fdc1d99415cd72519c9c3ac36f5e8b69428ea0f2.

10 months agoUpdate: Next micro version (0.6.9).
Kevin Day [Mon, 1 Jan 2024 16:36:00 +0000 (10:36 -0600)]
Update: Next micro version (0.6.9).

10 months agoCleanup: Fix incorrect endif disable macro comment. 0.6.8
Kevin Day [Sun, 31 Dec 2023 03:24:12 +0000 (21:24 -0600)]
Cleanup: Fix incorrect endif disable macro comment.

10 months agoUpdate: Explicitly cast UTF conversion to/from to a uint32_t.
Kevin Day [Sun, 31 Dec 2023 02:33:33 +0000 (20:33 -0600)]
Update: Explicitly cast UTF conversion to/from to a uint32_t.

If the type is changed or the defines are used with different types, then the shift operators may become problematic.
Prevent this potential problem from happening through explicit casts to uint32_t.

11 months agoBugfix: Fix incorrect wording regarding negative times and add some clarification...
Kevin Day [Sat, 16 Dec 2023 01:37:50 +0000 (19:37 -0600)]
Bugfix: Fix incorrect wording regarding negative times and add some clarification regarding defaults.

The time specification is mistakenly using the word "after" when it should be the word "before" for the example "2022:-5".

I decided that I needed to better clarify what I meant by default and not make it sound so strict.
Add additional language to better communicate that when the year is not specified, it is not specified.
There is no required interpretation and a default is simply a recommendation.

I am going to start trying to establish a habit of using UTC for my timestamps.
My local time zone, the date of the specification is still the 15 of December, 2023.
However, to stick with UTC for records purpose I need to the start doing this now.
There may be occasional inconsistencies for a while until I establish the habit of using UTC.

11 months agoUpdate: The fss-000e (Payload) specification should allow for the "payload" Section...
Kevin Day [Fri, 8 Dec 2023 02:29:47 +0000 (20:29 -0600)]
Update: The fss-000e (Payload) specification should allow for the "payload" Section to be optional.

In cases where packets that are being sent only need to send the "header", then it makes sense to not have a "payload" Section.
This should help reduce network traffic by avoiding the extra "payload:\n" characters when the payload Section is to be ignored.

12 months agoUpdate: Specifications for Payload and Simple Packet.
Kevin Day [Wed, 15 Nov 2023 00:00:37 +0000 (18:00 -0600)]
Update: Specifications for Payload and Simple Packet.

Add "type" as a recommended Object in the header section for the Payload.

Change the size units being used in Simple Packet.
The numbers used in some areas are completely wrong 2^4 is not 32-bits.
Instead use the word "bytes" or "bits" and update the units.
The Payload Block Structure uses "bytes" rather than bits as that is the general focus of that section.
All others use "bits".

Additional changes from previous updates in the 0.7 are added here as well (version 2023/10/04).

14 months agoUpdate: Use size_t and perform explicit casts in Control program.
Kevin Day [Sun, 10 Sep 2023 22:24:20 +0000 (17:24 -0500)]
Update: Use size_t and perform explicit casts in Control program.

The f_socket_read() function expects a size_t type but is passed f_array_length_t.
By default these should be the same thing.

Change "length" to a size_t and add appropriate casts.

This is not a very important change because the Control program is incomplete.

14 months agoUpdate: Next micro version (0.6.8).
Kevin Day [Thu, 7 Sep 2023 05:25:58 +0000 (00:25 -0500)]
Update: Next micro version (0.6.8).

14 months agoUpdate: Next micro version (0.6.7), for stand alone builds. 0.6.7
Kevin Day [Thu, 7 Sep 2023 02:37:03 +0000 (21:37 -0500)]
Update: Next micro version (0.6.7), for stand alone builds.

I missed these in the previous commit bumping the version number up.

14 months agoCleanup: Add comment in fll_execute_arguments_add_parameter_set() help to prevent...
Kevin Day [Thu, 7 Sep 2023 02:27:25 +0000 (21:27 -0500)]
Cleanup: Add comment in fll_execute_arguments_add_parameter_set() help to prevent possible misunderstandings in the future.

This is a back port of this comment from the 0.7 branch.

14 months agoCleanup: Relocate assignment in the if condition to avoid warning.
Kevin Day [Thu, 7 Sep 2023 02:05:32 +0000 (21:05 -0500)]
Cleanup: Relocate assignment in the if condition to avoid warning.

I think I was experiment with this style anyway.

Just move the assignment outside of the if condition.

14 months agoBugfix: Problems exposed by clang-13.
Kevin Day [Wed, 6 Sep 2023 05:45:46 +0000 (00:45 -0500)]
Bugfix: Problems exposed by clang-13.

The Vedic Extensions Unicode checks are out of order.

Remove unused "last" variable from private_fl_utf_file_write_until().

Add missing "return" keyword in Featureless Make.

14 months agoBugfix: Mistakes in the FSS-000f Simple Packet specification.
Kevin Day [Sat, 2 Sep 2023 14:27:31 +0000 (09:27 -0500)]
Bugfix: Mistakes in the FSS-000f Simple Packet specification.

The "Size Block Structure" section has the wrong example.
Fix the example to display the Size Block structure, which is a 32-bit digit.

Change the "Control Block Structure" section to use "size" annotations rather than bit examples.

14 months agoSecurity: Invalid allocation size for fll_execute_arguments_add_parameter_set().
Kevin Day [Mon, 28 Aug 2023 03:51:19 +0000 (22:51 -0500)]
Security: Invalid allocation size for fll_execute_arguments_add_parameter_set().

The size should be size * 2 because of the parameter and value are added.

14 months agoFeature: Add -U/--under parameter to Featureless Make.
Kevin Day [Mon, 28 Aug 2023 02:56:01 +0000 (21:56 -0500)]
Feature: Add -U/--under parameter to Featureless Make.

I discovered that the data build directory customization is incomplete while working on the Kevux Tools TacocaT program.

A new parameter to specify a sub-directory within the data directory, called "under", represented by -U and --under is now added.
This provides the ability to change the "build" part of the "data/build" directory.
This can even be an empty string.

The fakefile documentation and specification is updated because this parameter must now be reserved and exposed to IKI expansion.
I have not yet done a thorough test of this feature and I will hopefully do this before the next release.
I have performed a basic test and I can now compile the TacocaT program easily using this parameter.

14 months agoRegression: Incorrect dependencies breaking some builds, specifically the stand alone...
Kevin Day [Sun, 27 Aug 2023 21:17:44 +0000 (16:17 -0500)]
Regression: Incorrect dependencies breaking some builds, specifically the stand alone builds.

I'm not sure when this happened but at a some point I cleaned up some of the dependencies but failed to perform the appropraite checks with the stand alone builds.
There are stale files, dependencies, and references that are causing the stand alone builds to fail.

Remove or add the correct dependencies as appropriate.

15 months agoBugfix: Incorrect function name in error message.
Kevin Day [Wed, 23 Aug 2023 04:15:08 +0000 (23:15 -0500)]
Bugfix: Incorrect function name in error message.

15 months agoCleanup: Remove duplicate headers present in the wrong file.
Kevin Day [Sat, 12 Aug 2023 21:48:26 +0000 (16:48 -0500)]
Cleanup: Remove duplicate headers present in the wrong file.

This is likely a result of an incomplete relocation of these functions to the private file.
They are not needed and removing them neither breaks API nor ABI.

15 months agoCleanup: Use '{ 0 }' as initializer for f_thread_set_t_initialize.
Kevin Day [Fri, 11 Aug 2023 01:23:18 +0000 (20:23 -0500)]
Cleanup: Use '{ 0 }' as initializer for f_thread_set_t_initialize.

The f_thread_attribute_t_initialize works using '{ 0 }' as the initializer.
For whatever reason, the compilers do not like explicit initializers with the auto NULL ones like '{ 0 }'.
Switching to '{ 0 }' makes compilers happy and making compilers happy is often a good thing.

15 months agoCleanup: Change structure of fll_execute_program() checks.
Kevin Day [Thu, 10 Aug 2023 04:31:26 +0000 (23:31 -0500)]
Cleanup: Change structure of fll_execute_program() checks.

This is a follow up to the commit 1b43ceefc20836137ed95e19d275dda44d8a678d.

The previous commit is passing NULL to strrchr().
This should be fine, but does produce a warning.

Clean up the code and play it safe by changing the design to only call strrchr() when the value passed would be non-NULL.

15 months agoSecurity: Add additional checks against the parameters in fll_execute_program().
Kevin Day [Tue, 8 Aug 2023 01:19:00 +0000 (20:19 -0500)]
Security: Add additional checks against the parameters in fll_execute_program().

15 months agoCleanup: Minor code optimizations in fll_program.
Kevin Day [Thu, 3 Aug 2023 01:29:07 +0000 (20:29 -0500)]
Cleanup: Minor code optimizations in fll_program.

The context sets are being set to f_color_set_empty_s.
Have each set be assigned the return value of the other.

Change the status return structure for simple return conditions at the end of the functions.

15 months agoBugfix: Add missing parameter checking to fll_program_* functions, add "const" to...
Kevin Day [Thu, 3 Aug 2023 01:22:11 +0000 (20:22 -0500)]
Bugfix: Add missing parameter checking to fll_program_* functions, add "const" to parameter pointers, and add missing documentation comment for parameter.

16 months agoBugfix: Byte Dump program binary mode is printing wrong number of padding.
Kevin Day [Thu, 27 Jul 2023 02:37:44 +0000 (21:37 -0500)]
Bugfix: Byte Dump program binary mode is printing wrong number of padding.

16 months agoCleanup: Message in byte dump documenation comment.
Kevin Day [Wed, 26 Jul 2023 04:20:28 +0000 (23:20 -0500)]
Cleanup: Message in byte dump documenation comment.

16 months agoCleanup: Fix documentation comments to correctly reference project UTF.
Kevin Day [Sun, 23 Jul 2023 19:51:25 +0000 (14:51 -0500)]
Cleanup: Fix documentation comments to correctly reference project UTF.

16 months agoSecurity: The f_iki allocation functions are calling f_memory_destroy() and f_memory_...
Kevin Day [Sat, 22 Jul 2023 15:33:12 +0000 (10:33 -0500)]
Security: The f_iki allocation functions are calling f_memory_destroy() and f_memory_delete() when they should not, causing memory leak.

The private_f_iki_datas_adjust() should be called instead of f_memory_destroy().
The private_f_iki_datas_resize() should be called instead of f_memory_delete().

This is because the iki_datass needs the individual sets within to be deallocated.
De-allocating at the top level via the f_memory_destroy() and f_memory_delete() is very probably a memory leak.

16 months agoCleanup: Remove useless NULL assignment to pointer.
Kevin Day [Sat, 22 Jul 2023 01:09:23 +0000 (20:09 -0500)]
Cleanup: Remove useless NULL assignment to pointer.

This doesn't change the value of the pointer, so it should be a useless operation.

16 months agoBugfix: Incorrect sizeof() values for numerous type array functions.
Kevin Day [Sat, 22 Jul 2023 00:59:18 +0000 (19:59 -0500)]
Bugfix: Incorrect sizeof() values for numerous type array  functions.

The wrong sizeof types are being used.
This is likely a copy and paste error.

16 months agoBugfix: Incorrect sizeof() values for f_thread_mutex_attribute functions and f_thread...
Kevin Day [Fri, 21 Jul 2023 02:57:50 +0000 (21:57 -0500)]
Bugfix: Incorrect sizeof() values for f_thread_mutex_attribute functions and f_thread_lock functions.

The wrong sizeof types are being used.
This is likely a copy and paste error.

16 months agoFeature: Add specification FSS-0010 "Encrypted Simple Packet".
Kevin Day [Sun, 16 Jul 2023 22:52:42 +0000 (17:52 -0500)]
Feature: Add specification FSS-0010 "Encrypted Simple Packet".

This is an encrypted variation of the FSS-000F "Simple Packet" specification.
The third control bit (from the left) represents that the Simple Packet is encrypted.

The requirements for the format of the actual contents of Payload Block is further relaxed due encrypted data needing to be private (for what should be obvious reasons).

16 months agoUpdate: FSS-000f specification, adding reasoning and improve the wording.
Kevin Day [Sun, 16 Jul 2023 22:48:54 +0000 (17:48 -0500)]
Update: FSS-000f specification, adding reasoning and improve the wording.

Add a simple paragraph about the reason why the size bit includes the Control Block and the Size Block in its size representation.

Fix a mistake in the words where the standard has "...of that that...".
The back to back use of "that" is confusing and probably a typo.

16 months agoCleanup: Remove extra space in documentation comment of the standard.
Kevin Day [Sun, 16 Jul 2023 22:43:25 +0000 (17:43 -0500)]
Cleanup: Remove extra space in documentation comment of the standard.

This does not alter the standard itself and so the version date is left unchanged.

16 months agoUpdate: Specifications, adding explicit version date and change license line format.
Kevin Day [Sat, 15 Jul 2023 03:02:54 +0000 (22:02 -0500)]
Update: Specifications, adding explicit version date and change license line format.

Add a version, which contains the date of the last change to each of the specifications.
There will eventually be a specification version number such as "1.2.3".

The license line in the comments at the top of each specification is of this form: "license: ".
Remove the colon so that the comment, if extracted, could be processed as something simple like FSS-0000.

16 months agoUpdate: Specifications, replacing "emphasis" with "italic".
Kevin Day [Sat, 15 Jul 2023 02:51:30 +0000 (21:51 -0500)]
Update: Specifications, replacing "emphasis" with "italic".

16 months agoUpdate: FSS-000F (Simple Packet) Specification.
Kevin Day [Sat, 15 Jul 2023 02:49:29 +0000 (21:49 -0500)]
Update: FSS-000F (Simple Packet) Specification.

I have decided that the endianness bit must be first.
The String/Binary bit is now the second bit.
The remaining 6 Control bits are left undefined.

Do some cleanup up in the text.
Improve the language.
Make sure "italic" is used instead of "emphasis".

16 months agoCleanup: Upper case the first word of each sentence in documentation comments and...
Kevin Day [Thu, 13 Jul 2023 03:29:32 +0000 (22:29 -0500)]
Cleanup: Upper case the first word of each sentence in documentation comments and add missing lines in man pages.

The man pag generation processed decided to arbitrarily strip out parts of the documentation.
Add back the missing lines that I noticed are missing.

16 months agoUpdate: The defines in cygwin.txt documentation are out of date.
Kevin Day [Thu, 6 Jul 2023 01:29:40 +0000 (20:29 -0500)]
Update: The defines in cygwin.txt documentation are out of date.

16 months agoBugfix: Incorrect version numbers in some of the extra projects files.
Kevin Day [Thu, 6 Jul 2023 01:25:02 +0000 (20:25 -0500)]
Bugfix: Incorrect version numbers in some of the extra projects files.

This is a mistake in previous commits where I accidentally changed version numbers of these files.

16 months agoUpdate: Next micro version (0.6.7).
Kevin Day [Thu, 6 Jul 2023 01:24:37 +0000 (20:24 -0500)]
Update: Next micro version (0.6.7).

16 months agoUpdate: Add stand alone fss_basic_read, fss_basic_write, fss_extended_read, and fss_e... 0.6.6
Kevin Day [Sun, 2 Jul 2023 05:42:05 +0000 (00:42 -0500)]
Update: Add stand alone fss_basic_read, fss_basic_write, fss_extended_read, and fss_extended_write.

I took the lazy approach and copied the fss_payload_read and fss_payload_write.
These projects will have most of the same dependencies with only a few changes.

This lazy approach means that I may have included unnecessary data.

16 months agoBugfix: FSS Identify stand alone requires f_string_dynamic_append_nulless.
Kevin Day [Sun, 2 Jul 2023 05:21:38 +0000 (00:21 -0500)]
Bugfix: FSS Identify stand alone requires f_string_dynamic_append_nulless.

16 months agoCleanup: Do not include man page documentation if there are no man pages.
Kevin Day [Sun, 2 Jul 2023 04:09:07 +0000 (23:09 -0500)]
Cleanup: Do not include man page documentation if there are no man pages.

This never generated an error locally because the directories happen to exist.
Empty directories are generally not stored in git and so when unit tests were run on Github the missing directory problem becomes exposed.

This is not a bug because the related bug is fixed in commit ecddd4d16504d3e2d9502d6a4dfe67658dce98dc.
This only silences the newly added warning.

16 months agoBugfix: Use more correct include paths in the man pages.
Kevin Day [Sun, 2 Jul 2023 04:07:44 +0000 (23:07 -0500)]
Bugfix: Use more correct include paths in the man pages.

The generated part of the man pages is using the wrong include headers.

This is a follow up to the commit 63be746d22d66870ae89f9575c82db7d0b784e32 where I expected to have problems.
At least I stumbled on this problem rather quickly.

16 months agoBugfix: The boostrap script should not fail if documentation directory is empty.
Kevin Day [Sun, 2 Jul 2023 04:06:31 +0000 (23:06 -0500)]
Bugfix: The boostrap script should not fail if documentation directory is empty.

Instead of failing, print a warning and continue onward.

16 months agoUpdate: Manually add missing man pages.
Kevin Day [Sun, 2 Jul 2023 03:56:13 +0000 (22:56 -0500)]
Update: Manually add missing man pages.

I don't know why these were missed during the generation process.

I copied another man page and modified it accordingly, attempting to keep to a similar structure.

16 months agoUpdate: Add man pages for most or all FLL functions.
Kevin Day [Sun, 2 Jul 2023 02:38:33 +0000 (21:38 -0500)]
Update: Add man pages for most or all FLL functions.

This is a huge amount of work.
I attempted to use several tools and doxygen had the best results.
The doxygen did a tolerable job but did not do a good job.
In fact, the man page generation of doxygen is horrible.
I instead used doxygen to create XML and then use a tool called doxy2man.
The end result is a barely tolerable man page.

I then had to go through and mass change the files due to problems.
I've brute-force stripped out a lot of messed up parts rather than try to address them.
Some of the data I did manually add or tweak and this task is incredibly tedius.

I observed that a lot of the documentation regarding function returns never completely made it into the man pages.
This means that the man pages are incomplete.
This is, however, better than nothing.

In the long term I can see myself writing my own parser (which makes me cringe).
I believe that I need to create a man page alternative program that uses the FSS formats for a more sane and more readable format.
This is not happening now or any time soon.

I specifically used doxywizard for the doxygen and the following is the configuration settings with the comments removed.

DOXYFILE_ENCODING      = UTF-8
PROJECT_NAME           = FLL
PROJECT_NUMBER         = 0.6.6
PROJECT_BRIEF          =
PROJECT_LOGO           =
OUTPUT_DIRECTORY       = generated
CREATE_SUBDIRS         = NO
ALLOW_UNICODE_NAMES    = NO
OUTPUT_LANGUAGE        = English
BRIEF_MEMBER_DESC      = YES
REPEAT_BRIEF           = YES
ABBREVIATE_BRIEF       = "The $name class" \
                         "The $name widget" \
                         "The $name file" \
                         is \
                         provides \
                         specifies \
                         contains \
                         represents \
                         a \
                         an \
                         the
ALWAYS_DETAILED_SEC    = NO
INLINE_INHERITED_MEMB  = NO
FULL_PATH_NAMES        = YES
STRIP_FROM_PATH        =
STRIP_FROM_INC_PATH    =
SHORT_NAMES            = NO
JAVADOC_AUTOBRIEF      = NO
QT_AUTOBRIEF           = NO
MULTILINE_CPP_IS_BRIEF = NO
INHERIT_DOCS           = NO
SEPARATE_MEMBER_PAGES  = NO
TAB_SIZE               = 4
ALIASES                =
TCL_SUBST              =
OPTIMIZE_OUTPUT_FOR_C  = YES
OPTIMIZE_OUTPUT_JAVA   = NO
OPTIMIZE_FOR_FORTRAN   = NO
OPTIMIZE_OUTPUT_VHDL   = NO
EXTENSION_MAPPING      =
MARKDOWN_SUPPORT       = YES
TOC_INCLUDE_HEADINGS   = 0
AUTOLINK_SUPPORT       = YES
BUILTIN_STL_SUPPORT    = NO
CPP_CLI_SUPPORT        = NO
SIP_SUPPORT            = NO
IDL_PROPERTY_SUPPORT   = YES
DISTRIBUTE_GROUP_DOC   = NO
GROUP_NESTED_COMPOUNDS = NO
SUBGROUPING            = YES
INLINE_GROUPED_CLASSES = NO
INLINE_SIMPLE_STRUCTS  = NO
TYPEDEF_HIDES_STRUCT   = NO
LOOKUP_CACHE_SIZE      = 0
EXTRACT_ALL            = YES
EXTRACT_PRIVATE        = NO
EXTRACT_PACKAGE        = NO
EXTRACT_STATIC         = NO
EXTRACT_LOCAL_CLASSES  = YES
EXTRACT_LOCAL_METHODS  = NO
EXTRACT_ANON_NSPACES   = NO
HIDE_UNDOC_MEMBERS     = NO
HIDE_UNDOC_CLASSES     = NO
HIDE_FRIEND_COMPOUNDS  = NO
HIDE_IN_BODY_DOCS      = NO
INTERNAL_DOCS          = NO
CASE_SENSE_NAMES       = NO
HIDE_SCOPE_NAMES       = NO
HIDE_COMPOUND_REFERENCE= NO
SHOW_INCLUDE_FILES     = YES
SHOW_GROUPED_MEMB_INC  = NO
FORCE_LOCAL_INCLUDES   = NO
INLINE_INFO            = YES
SORT_MEMBER_DOCS       = YES
SORT_BRIEF_DOCS        = NO
SORT_MEMBERS_CTORS_1ST = NO
SORT_GROUP_NAMES       = NO
SORT_BY_SCOPE_NAME     = NO
STRICT_PROTO_MATCHING  = NO
GENERATE_TODOLIST      = NO
GENERATE_TESTLIST      = NO
GENERATE_BUGLIST       = NO
GENERATE_DEPRECATEDLIST= NO
ENABLED_SECTIONS       =
MAX_INITIALIZER_LINES  = 30
SHOW_USED_FILES        = YES
SHOW_FILES             = YES
SHOW_NAMESPACES        = YES
FILE_VERSION_FILTER    =
LAYOUT_FILE            =
CITE_BIB_FILES         =
QUIET                  = NO
WARNINGS               = YES
WARN_IF_UNDOCUMENTED   = YES
WARN_IF_DOC_ERROR      = YES
WARN_NO_PARAMDOC       = NO
WARN_AS_ERROR          = NO
WARN_FORMAT            = "$file:$line: $text"
WARN_LOGFILE           =
INPUT                  = includes
INPUT_ENCODING         = UTF-8
FILE_PATTERNS          = *.c \
                         *.cc \
                         *.cxx \
                         *.cpp \
                         *.c++ \
                         *.java \
                         *.ii \
                         *.ixx \
                         *.ipp \
                         *.i++ \
                         *.inl \
                         *.idl \
                         *.ddl \
                         *.odl \
                         *.h \
                         *.hh \
                         *.hxx \
                         *.hpp \
                         *.h++ \
                         *.cs \
                         *.d \
                         *.php \
                         *.php4 \
                         *.php5 \
                         *.phtml \
                         *.inc \
                         *.m \
                         *.markdown \
                         *.md \
                         *.mm \
                         *.dox \
                         *.py \
                         *.pyw \
                         *.f90 \
                         *.f95 \
                         *.f03 \
                         *.f08 \
                         *.f \
                         *.for \
                         *.tcl \
                         *.vhd \
                         *.vhdl \
                         *.ucf \
                         *.qsf
RECURSIVE              = YES
EXCLUDE                =
EXCLUDE_SYMLINKS       = NO
EXCLUDE_PATTERNS       =
EXCLUDE_SYMBOLS        =
EXAMPLE_PATH           =
EXAMPLE_PATTERNS       = *
EXAMPLE_RECURSIVE      = NO
IMAGE_PATH             =
INPUT_FILTER           =
FILTER_PATTERNS        =
FILTER_SOURCE_FILES    = NO
FILTER_SOURCE_PATTERNS =
USE_MDFILE_AS_MAINPAGE =
SOURCE_BROWSER         = NO
INLINE_SOURCES         = NO
STRIP_CODE_COMMENTS    = NO
REFERENCED_BY_RELATION = NO
REFERENCES_RELATION    = NO
REFERENCES_LINK_SOURCE = YES
SOURCE_TOOLTIPS        = YES
USE_HTAGS              = NO
VERBATIM_HEADERS       = YES
CLANG_ASSISTED_PARSING = NO
CLANG_OPTIONS          =
ALPHABETICAL_INDEX     = YES
COLS_IN_ALPHA_INDEX    = 5
IGNORE_PREFIX          =
GENERATE_HTML          = NO
HTML_OUTPUT            = html
HTML_FILE_EXTENSION    = .html
HTML_HEADER            =
HTML_FOOTER            =
HTML_STYLESHEET        =
HTML_EXTRA_STYLESHEET  =
HTML_EXTRA_FILES       =
HTML_COLORSTYLE_HUE    = 220
HTML_COLORSTYLE_SAT    = 100
HTML_COLORSTYLE_GAMMA  = 80
HTML_TIMESTAMP         = NO
HTML_DYNAMIC_SECTIONS  = NO
HTML_INDEX_NUM_ENTRIES = 100
GENERATE_DOCSET        = NO
DOCSET_FEEDNAME        = "Doxygen generated docs"
DOCSET_BUNDLE_ID       = org.doxygen.Project
DOCSET_PUBLISHER_ID    = org.doxygen.Publisher
DOCSET_PUBLISHER_NAME  = Publisher
GENERATE_HTMLHELP      = NO
CHM_FILE               =
HHC_LOCATION           =
GENERATE_CHI           = NO
CHM_INDEX_ENCODING     =
BINARY_TOC             = NO
TOC_EXPAND             = NO
GENERATE_QHP           = NO
QCH_FILE               =
QHP_NAMESPACE          = org.doxygen.Project
QHP_VIRTUAL_FOLDER     = doc
QHP_CUST_FILTER_NAME   =
QHP_CUST_FILTER_ATTRS  =
QHP_SECT_FILTER_ATTRS  =
QHG_LOCATION           =
GENERATE_ECLIPSEHELP   = NO
ECLIPSE_DOC_ID         = org.doxygen.Project
DISABLE_INDEX          = NO
GENERATE_TREEVIEW      = NO
ENUM_VALUES_PER_LINE   = 4
TREEVIEW_WIDTH         = 250
EXT_LINKS_IN_WINDOW    = NO
FORMULA_FONTSIZE       = 10
FORMULA_TRANSPARENT    = YES
USE_MATHJAX            = NO
MATHJAX_FORMAT         = HTML-CSS
MATHJAX_RELPATH        = http://cdn.mathjax.org/mathjax/latest
MATHJAX_EXTENSIONS     =
MATHJAX_CODEFILE       =
SEARCHENGINE           = YES
SERVER_BASED_SEARCH    = NO
EXTERNAL_SEARCH        = NO
SEARCHENGINE_URL       =
SEARCHDATA_FILE        = searchdata.xml
EXTERNAL_SEARCH_ID     =
EXTRA_SEARCH_MAPPINGS  =
GENERATE_LATEX         = NO
LATEX_OUTPUT           = latex
LATEX_CMD_NAME         = latex
MAKEINDEX_CMD_NAME     = makeindex
COMPACT_LATEX          = NO
PAPER_TYPE             = a4
EXTRA_PACKAGES         =
LATEX_HEADER           =
LATEX_FOOTER           =
LATEX_EXTRA_STYLESHEET =
LATEX_EXTRA_FILES      =
PDF_HYPERLINKS         = YES
USE_PDFLATEX           = YES
LATEX_BATCHMODE        = NO
LATEX_HIDE_INDICES     = NO
LATEX_SOURCE_CODE      = NO
LATEX_BIB_STYLE        = plain
LATEX_TIMESTAMP        = NO
GENERATE_RTF           = NO
RTF_OUTPUT             = rtf
COMPACT_RTF            = NO
RTF_HYPERLINKS         = NO
RTF_STYLESHEET_FILE    =
RTF_EXTENSIONS_FILE    =
RTF_SOURCE_CODE        = NO
GENERATE_MAN           = NO
MAN_OUTPUT             = man
MAN_EXTENSION          = .3
MAN_SUBDIR             =
MAN_LINKS              = NO
GENERATE_XML           = YES
XML_OUTPUT             = xml
XML_PROGRAMLISTING     = NO
GENERATE_DOCBOOK       = NO
DOCBOOK_OUTPUT         = docbook
DOCBOOK_PROGRAMLISTING = NO
GENERATE_AUTOGEN_DEF   = NO
GENERATE_PERLMOD       = NO
PERLMOD_LATEX          = NO
PERLMOD_PRETTY         = YES
PERLMOD_MAKEVAR_PREFIX =
ENABLE_PREPROCESSING   = YES
MACRO_EXPANSION        = YES
EXPAND_ONLY_PREDEF     = NO
SEARCH_INCLUDES        = YES
INCLUDE_PATH           =
INCLUDE_FILE_PATTERNS  =
PREDEFINED             =
EXPAND_AS_DEFINED      =
SKIP_FUNCTION_MACROS   = YES
TAGFILES               =
GENERATE_TAGFILE       =
ALLEXTERNALS           = NO
EXTERNAL_GROUPS        = YES
EXTERNAL_PAGES         = YES
PERL_PATH              = /usr/bin/perl
CLASS_DIAGRAMS         = NO
MSCGEN_PATH            =
DIA_PATH               =
HIDE_UNDOC_RELATIONS   = YES
HAVE_DOT               = NO
DOT_NUM_THREADS        = 0
DOT_FONTNAME           = Helvetica
DOT_FONTSIZE           = 10
DOT_FONTPATH           =
CLASS_GRAPH            = YES
COLLABORATION_GRAPH    = YES
GROUP_GRAPHS           = YES
UML_LOOK               = NO
UML_LIMIT_NUM_FIELDS   = 10
TEMPLATE_RELATIONS     = NO
INCLUDE_GRAPH          = YES
INCLUDED_BY_GRAPH      = YES
CALL_GRAPH             = NO
CALLER_GRAPH           = NO
GRAPHICAL_HIERARCHY    = YES
DIRECTORY_GRAPH        = YES
DOT_IMAGE_FORMAT       = png
INTERACTIVE_SVG        = NO
DOT_PATH               =
DOTFILE_DIRS           =
MSCFILE_DIRS           =
DIAFILE_DIRS           =
PLANTUML_JAR_PATH      =
PLANTUML_CFG_FILE      =
PLANTUML_INCLUDE_PATH  =
DOT_GRAPH_MAX_NODES    = 50
MAX_DOT_GRAPH_DEPTH    = 0
DOT_TRANSPARENT        = NO
DOT_MULTI_TARGETS      = NO
GENERATE_LEGEND        = YES
DOT_CLEANUP            = YES

16 months agoBugfix: The build scripts are not handling documentation directory.
Kevin Day [Sun, 2 Jul 2023 02:35:40 +0000 (21:35 -0500)]
Bugfix: The build scripts are not handling documentation directory.

The man pages are not being correctly copied over, if at all.

16 months agoUpdate: Relax the requirement in the license to be less verbose.
Kevin Day [Thu, 29 Jun 2023 04:39:59 +0000 (23:39 -0500)]
Update: Relax the requirement in the license to be less verbose.

Do not specifically use my name "Kevin Day".
Instead refer to the "original license owner" or some authorized party.
This will make the license more flexible.

16 months agoUpdate: Change program man pages to be referred to as "Program Manual".
Kevin Day [Mon, 26 Jun 2023 22:58:21 +0000 (17:58 -0500)]
Update: Change program man pages to be referred to as "Program Manual".

This makes a little more sense than "User Commands".
There is still room for improvement but this is progress.

17 months agoBugfix: FSS Identify should process all --name parameters and not the last.
Kevin Day [Sun, 25 Jun 2023 21:12:09 +0000 (16:12 -0500)]
Bugfix: FSS Identify should process all --name parameters and not the last.

Back port a bug fix from the 0.7.x versions to make sure --name behaves more naturally.
All --name parameters should be checked for.

17 months agoBugfix: The fss_identify_process() should have f_string_empty_s rather than f_string_...
Kevin Day [Sun, 25 Jun 2023 02:32:17 +0000 (21:32 -0500)]
Bugfix: The fss_identify_process() should have f_string_empty_s rather than f_string_ascii_minus_s.

When the pipe is being used an empty string is expected rather than the minus character.

17 months agoBugfix: Remove spurious 'y' character in help message.
Kevin Day [Sun, 25 Jun 2023 02:19:00 +0000 (21:19 -0500)]
Bugfix: Remove spurious 'y' character in help message.

17 months agoCleanup: Fix documentation comments for fss_identify_process_name_compare().
Kevin Day [Sun, 25 Jun 2023 02:30:38 +0000 (21:30 -0500)]
Cleanup: Fix documentation comments for fss_identify_process_name_compare().

The wrong return values are being reported.
The fl_string_dynamic_partial_compare_string() function is missing.

17 months agoBugfix: Add missing 'extern' keyword for fss_identify_process_name_compare() definition.
Kevin Day [Sat, 24 Jun 2023 23:24:06 +0000 (18:24 -0500)]
Bugfix: Add missing 'extern' keyword for fss_identify_process_name_compare() definition.

17 months agoCleanup: Documentation comments return status syntax is missing a colon.
Kevin Day [Sat, 24 Jun 2023 22:35:38 +0000 (17:35 -0500)]
Cleanup: Documentation comments return status syntax is missing a colon.

17 months agoCleanup: Add quotes to testfile LD_LIBRARY_PATH assignment.
Kevin Day [Sat, 24 Jun 2023 20:25:27 +0000 (15:25 -0500)]
Cleanup: Add quotes to testfile LD_LIBRARY_PATH assignment.

Do this for consistency with the pattern of the rest of the testfile.

17 months agoCleanup: Add missing () in docuumentation comments for f_file_seek().
Kevin Day [Sat, 24 Jun 2023 20:21:59 +0000 (15:21 -0500)]
Cleanup: Add missing () in docuumentation comments for f_file_seek().

17 months agoCleanup: Fix mistakes in documentation comments.
Kevin Day [Sat, 24 Jun 2023 03:14:46 +0000 (22:14 -0500)]
Cleanup: Fix mistakes in documentation comments.

These are back ported clean ups discovered while developing on the 0.7.x version.

17 months agoCleanup: Add environment to all unit test settings files.
Kevin Day [Sat, 24 Jun 2023 02:44:12 +0000 (21:44 -0500)]
Cleanup: Add environment to all unit test settings files.

17 months agoCleanup: Fix incorrect comment in f_limit.
Kevin Day [Wed, 21 Jun 2023 02:07:38 +0000 (21:07 -0500)]
Cleanup: Fix incorrect comment in f_limit.

17 months agoUpdate: The Open Standard License, adding stronger terminology and more clarification.
Kevin Day [Sun, 18 Jun 2023 03:42:25 +0000 (22:42 -0500)]
Update: The Open Standard License, adding stronger terminology and more clarification.

17 months agoBugfix: FSS Extended List Read program should print new line after Object.
Kevin Day [Fri, 16 Jun 2023 02:39:55 +0000 (21:39 -0500)]
Bugfix: FSS Extended List Read program should print new line after Object.

In the case where Object is printed but not Content a new line must be printed.
This is done to be consistent with the other programs, such as FSS Basic List Read.

The 0.7.x and later versions should probably implement something like -L/--line_end to perform this behavior.
In which case the default behavior would be what the code was doing here prior to this commit.

17 months agoCleanup: Minor syntax fixes and use the macro initializer.
Kevin Day [Fri, 16 Jun 2023 02:03:31 +0000 (21:03 -0500)]
Cleanup: Minor syntax fixes and use the macro initializer.

17 months agoBugfix: The line and total need to support the closing brace.
Kevin Day [Fri, 16 Jun 2023 00:33:20 +0000 (19:33 -0500)]
Bugfix: The line and total need to support the closing brace.

The closing brace is now being calculated when both --object and --content are specified.

17 months agoBugfix: Use arguments passed to fake when processing build with the custom build...
Kevin Day [Fri, 9 Jun 2023 04:17:26 +0000 (23:17 -0500)]
Bugfix: Use arguments passed to fake when processing build with the custom build arguments.

The custom build arguments are passed when the make mode is being used to call the build mode.
The mode arguments passed to the fake program, these should be propagated into the custom build arguments for the build mode.

Move the print message to after the build modes are processed so that an accurate message is printed.

17 months agoBugfix: Use the custom build settings name if passed when printing current setting...
Kevin Day [Fri, 9 Jun 2023 03:09:24 +0000 (22:09 -0500)]
Bugfix: Use the custom build settings name if passed when printing current setting being used.

The message, such as:
  Building fss_read using 'settings' with modes 'monolithic', 'thread'.

Should instead be:
  Building fss_read using 'settings.main' with modes 'monolithic', 'thread'.

Use the custom settings name if the custom build arguments are passed to display the correct name.

17 months agoBugfix: Quotes in FSS read functions are not being properly handled and fix backtick...
Kevin Day [Wed, 31 May 2023 04:31:13 +0000 (23:31 -0500)]
Bugfix: Quotes in FSS read functions are not being properly handled and fix backtick handling.

At some point in time the intended behavior changed from using the literal quote character to an enumeration representing the quote character on FSS read calls.
This transition is incomplete and broken.

Finish the transition, passing the enumeration representation rather than the literal character.

Remove an unnecessary initialization on the quote because the private function being called also initializes the variable.

Relocate the f_uint8s_increase() call to be after changing the error status to ensure that the quotes array is properly incremented.

Update the documentation comments to better clarify this behavior in regards to the enumeration.

There are several FSS read functions that are not handling backticks properly.
The backticks should now be fully and properly processed on read.

17 months agoBugfix: Handle f_directory_not status as an invalid directory.
Kevin Day [Mon, 29 May 2023 23:22:57 +0000 (18:22 -0500)]
Bugfix: Handle f_directory_not status as an invalid directory.

The status f_directory_not is returned with the path exists but is not a directory.

The fake program can fail when doing something like this:
  fake clean make -w /arbitrary_non_directory_file +V

The error message returned is not very helpful:
  ERROR: (674) when calling function f_file_stat().

This is a known file-related error and so it is now handled appropraitely.
The new error message looks like:
  ERROR: Invalid directory while trying to access directory '/arbitrary_non_directory_file'.

17 months agoBugfix: The testfile needs to handle the --build parameter and improve --work paramet...
Kevin Day [Mon, 29 May 2023 23:16:44 +0000 (18:16 -0500)]
Bugfix: The testfile needs to handle the --build parameter and improve --work parameter handling.

Provide a new operation "build_path" to load the "--build" parameter properly.
The existing checks for the "--work" parameter is better suited checking against "work:value" rather than "work".

There may be an existing bug where the condition for "build" parameter is always returning true when it should not.
This requires further investigation and is not addressed within the scope of this commit.