URL: https://github.com/freeipa/freeipa/pull/268
Title: #268: Build system must regenerate file when template changes
mbasti-rh commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/ba6ae666acaf8b930d18f45efc7c9c9faad3526b
https://fedorahosted.org/freeipa/changeset/685
URL: https://github.com/freeipa/freeipa/pull/268
Title: #268: Build system must regenerate file when template changes
pspacek commented:
"""
Oh, you are right, I was mixing `CONFIG_STATUS_DEPEDENCIES` and
`AC_CONFIG_FILES`. Sorry!
So please let me explain the problem with `AC_CONFIG_FILES`:
`A
URL: https://github.com/freeipa/freeipa/pull/268
Title: #268: Build system must regenerate file when template changes
tiran commented:
"""
You gave a good reason to not use ```CONFIG_STATUS_DEPEDENCIES``` and I agree
with your reasoning. I don't see a case against ```AC_CONFIG_FILE```, though.
URL: https://github.com/freeipa/freeipa/pull/268
Title: #268: Build system must regenerate file when template changes
pspacek commented:
"""
I already described problems with `AC_CONFIG_FILE` in
https://github.com/freeipa/freeipa/pull/251#issuecomment-261470338 a week ago,
including envisioned
URL: https://github.com/freeipa/freeipa/pull/268
Title: #268: Build system must regenerate file when template changes
tiran commented:
"""
I don't like the approach and prefer ```AC_CONFIG_FILE``` over manual sed for
templating. You only have to add a couple of rules like
```
# Makefile.pytho
URL: https://github.com/freeipa/freeipa/pull/268
Title: #268: Build system must regenerate file when template changes
mbasti-rh commented:
"""
works for me
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/268#issuecomment-262939243
--
Manage your subscription for the Freeip
