Expand environment variables in some configuration variable

[Konubinix]

No description provided.

[konvpalto]

At which use-case these modification are targeted? You need to expand some other variable that $HOME?

[Konubinix]

Hi,
konvpalto notifications@github.com writes:

At which use-case these modification are targeted? You need to expand some other variable that $HOME?
For the time being, I have
--8<---------------cut here---------------start------------->8---
pythonfile = ${KONIX_PERSO_DIR}/offlineimap.py
--8<---------------cut here---------------end--------------->8---

and
--8<---------------cut here---------------start------------->8---
sslcacertfile = ${KONIX_CA_FILE}
--8<---------------cut here---------------end--------------->8---

In my offlineimap.conf file.

Location pointed by those environment variable depend on the machine I
am working on.

In order to only expand the HOME variable, I would have to move a lot of
files to have the same file hierarchy on all the computers.

Konubinix
GPG Key : 7439106A
Fingerprint: 5993 BE7A DA65 E2D9 06CE 5C36 75D2 3CED 7439 106A

[spaetz]

That having said, I defer the decision to merge this to konvpalto who should be making the call.

[konvpalto]

I'll merge this feature, but I am planning to have more generic mechanism to have filters (read, environment variable expansion and other stuff) to be done not in every function, but to make it driven by configuration hash inside the OI code. So it will take some time, though I don't expect that it will be more than a week.

[Konubinix]

I can't wait to see it :-)

[Ram-Z]

I was about to start writing a patch for this. My use case is very similar:

pythonfile = $XDG_CONFIG_HOME/offlineimap/utils.py
metadata = $XDG_DATA_HOME/offlineimap

[Konubinix]

Hi,
konvpalto notifications@github.com writes:

I'll merge this feature, but I am planning to have more generic mechanism to have filters (read, environment variable expansion and other stuff) to be done not in every function, but to make it driven by configuration hash inside the OI code. So it will take some time, though I don't expect that it will be more than a week.

Do you still plan to do it.

May be you can tell us exactly what should filters look like. I would be happy to write
the code.

Konubinix
GPG Key : 7439106A
Fingerprint: 5993 BE7A DA65 E2D9 06CE 5C36 75D2 3CED 7439 106A

[brandur]

For the little it's worth, +1 for pulling this in for the interim until the more sophisticated filtering system can about.

[konvpalto]

Sorry, the patch was living in my tree for a couple of weeks, but was drowned with $WORK. 8f3c22e and e51ed80 that were just been pushed to "next" should implement the needed functionality.

Please, test.

[Konubinix]

konvpalto notifications@github.com writes:

Sorry, the patch was living in my tree for a couple of weeks, but was drowned with $WORK. 8f3c22e and e51ed80 that were just been pushed to "next" should implement the needed functionality.

Please, test.

It appears to work. The code is indeed much cleaner than what I
proposed. Thank you for taking this use case into account.

Konubinix
GPG Key : 7439106A
Fingerprint: 5993 BE7A DA65 E2D9 06CE 5C36 75D2 3CED 7439 106A

[konvpalto]

Thanks a lot for your testing and proposed functionality!