View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004555 | GNUnet | util library | public | 2016-06-01 22:29 | 2018-06-06 22:58 |
| Reporter | tg | Assigned To | Christian Grothoff | Due Date | |
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | no change required | ||
| Product Version | Git master | ||||
| Fixed in Version | 0.11.0pre66 | ||||
| Summary | 0004555: config variable expansion not working | ||||
| Description | Expansion of variables defined in the [PATH] section does not always work. It works in the [arm] section in GLOBAL_POSTFIX: [arm] GLOBAL_POSTFIX = -l $GNUNET_CACHE_HOME/{}-%F.%T-[].log But not in other service sections when used in OPTIONS: [social] OPTIONS = -L DEBUG -l $GNUNET_CACHE_HOME/{}-%F.%T-[].log --- UPDATE: after the fix for either 0004545 or 0004546 it does not work anymore in either case. | ||||
| Tags | No tags attached. | ||||
| I've tested it, $-expansion works for me as expected. | |
| double-checked with the config above, definitely not working here. a ~/$GNUNET_CACHE_HOME (literally like this) directory is created when I put it in either GLOBAL_POSTFIX or OPTIONS | |
| Doesn't seem to work properly in the [arm] section here, with gnunet 0.10.1: ----gnunet.conf---- [PATHS] GNUNET_HOME = ~/gnunet_peer2/ [arm] DEFAULTSERVICES = UNIXPATH = $GNUNET_USER_RUNTIME_DIR/gnunet-service-armi-p2.sock GLOBAL_POSTFIX = -L WARNING -l $GNUNET_HOME/gnunet.log ----eof---- This leads to the "-l _HOME/gnunet.log" option. | |
| Ah, but note that we do not generally expand all options just because they contain a "$". We ONLY do the $-expansion if the application expects a FILENAME, i.e. if GNUNET_CONFIGURATION_get_value_filename() is used. If the code uses GNUNET_CONFIGURATION_get_value_string(), $-expansion is *intentionally* not applied, as doing so can also break things (TM). So really the issue here is a missmatch between what you expect (and it is perfectly reasonable to expect $-expansion for GLOBAL_POSTFIX), and what configuration does by definition. I *think* we might revise what configuration does by definition a bit, but it's tricky as $-expansion does more than substituting $PATHs, and so blinding doing it will break more than it fixes. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2016-06-01 22:29 | tg | New Issue | |
| 2016-06-01 22:29 | tg | Assigned To | => Christian Grothoff |
| 2016-06-01 22:29 | tg | Status | new => assigned |
| 2016-06-02 15:28 | Christian Grothoff | Note Added: 0010850 | |
| 2016-06-02 15:29 | Christian Grothoff | Assigned To | Christian Grothoff => tg |
| 2016-06-02 15:29 | Christian Grothoff | Status | assigned => feedback |
| 2016-06-02 15:29 | Christian Grothoff | Product Version | => Git master |
| 2016-06-02 15:29 | Christian Grothoff | Target Version | => 0.11.0pre66 |
| 2016-06-02 23:27 | tg | Note Added: 0010853 | |
| 2016-06-02 23:27 | tg | Status | feedback => assigned |
| 2016-07-08 19:33 | defanor | Note Added: 0010981 | |
| 2017-02-21 17:43 | Christian Grothoff | Note Added: 0011821 | |
| 2017-02-21 17:43 | Christian Grothoff | Assigned To | tg => Christian Grothoff |
| 2017-02-26 00:34 | Christian Grothoff | Assigned To | Christian Grothoff => |
| 2017-02-26 00:34 | Christian Grothoff | Status | assigned => acknowledged |
| 2017-02-26 00:34 | Christian Grothoff | Target Version | 0.11.0pre66 => |
| 2018-06-06 22:58 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2018-06-06 22:58 | Christian Grothoff | Status | acknowledged => closed |
| 2018-06-06 22:58 | Christian Grothoff | Resolution | open => no change required |
| 2018-06-06 22:58 | Christian Grothoff | Fixed in Version | => 0.11.0pre66 |