Skip to content
Permalink
Browse files

Merge pull request #2864 from kodebach/fixes

Fixes for gopts
  • Loading branch information...
markus2330 committed Aug 9, 2019
2 parents 2edf84c + f44080b commit 7bb89771ea74b22ce0ad19997db670fbe7be12b5
Showing with 6 additions and 5 deletions.
  1. +3 −4 doc/news/_preparation_next_release.md
  2. +2 −1 src/plugins/gopts/CMakeLists.txt
  3. +1 −0 src/plugins/gopts/gopts.c
@@ -39,11 +39,10 @@ You can also read the news [on our website](https://www.libelektra.org/news/0.9.

The following section lists news about the [modules](https://www.libelektra.org/plugins/readme) we updated in this release.

### <<Plugin1>>
### GOpts

- <<TODO>>
- <<TODO>>
- <<TODO>>
- The error message, if non of the gopts variants can be compiled, was improved. _(Klemens Böswirth)_
- A better error, if the plugin fails to load `argv` from the system, was added. _(Klemens Böswirth)_

### <<Plugin2>>

@@ -17,7 +17,8 @@ else ()
set (ELEKTRA_GOPTS_PROCFS ON)
set (gopts_source gopts_procfs.h)
else ()
message (STATUS "gopts: no procfs and sysctl test failed with:")
message (STATUS "gopts: Failed to find one of the supported platforms. Supported are the Win32 API, the OSX API, the "
"sysctl API (with support for KERN_PROC_ARGS) or a mounted procfs.")
message ("${SYSCTL_TEST_OUTPUT}")
endif ()
endif ()
@@ -57,6 +57,7 @@ int elektraGOptsGet (Plugin * handle ELEKTRA_UNUSED, KeySet * returned, Key * pa

if (argv == NULL || envp == NULL)
{
ELEKTRA_SET_INSTALLATION_ERROR (parentKey, "could not load current process' arguments");
return ELEKTRA_PLUGIN_STATUS_ERROR;
}

0 comments on commit 7bb8977

Please sign in to comment.
You can’t perform that action at this time.