7:40 p.m.
Attached patch fixes #1260: Installation fails due to double hyphen
"--" in hostname.
Cheers,
Fraser
12:24 p.m.
On 3/2/2016 10:04 PM, Fraser Tweedale wrote:
On Thu, Mar 03, 2016 at 11:40:15AM +1000, Fraser Tweedale wrote:
> Attached patch fixes #1260: Installation fails due to double hyphen
> "--" in hostname.
>
> Cheers,
> Fraser
>
ACKed by Ade;
pushed to master (8beb5cfa4cd81fbf47ea8cd6839b793c2a12284e)
Cheers,
Fraser
The patch changes "<!-- -->" into "<?pkidaemon ?>" in new
instances
only. Suppose in the future we write a script that parses "<?pkidaemon
?>" (e.g. to remove the section) it would not be able to find it in an
upgraded instance. Ideally there should be an upgrade script to change
"<!-- -->" into "<?pkidaemon ?>" in existing instances
too.
--
Endi S. Dewata
6:43 p.m.
On Tue, Mar 15, 2016 at 12:24:58PM -0500, Endi Sukma Dewata wrote:
On 3/2/2016 10:04 PM, Fraser Tweedale wrote:
>On Thu, Mar 03, 2016 at 11:40:15AM +1000, Fraser Tweedale wrote:
>>Attached patch fixes #1260: Installation fails due to double hyphen
>>"--" in hostname.
>>
>>Cheers,
>>Fraser
>>
>ACKed by Ade;
>pushed to master (8beb5cfa4cd81fbf47ea8cd6839b793c2a12284e)
>
>Cheers,
>Fraser
The patch changes "<!-- -->" into "<?pkidaemon ?>" in new
instances only.
Suppose in the future we write a script that parses "<?pkidaemon ?>"
(e.g.
to remove the section) it would not be able to find it in an upgraded
instance. Ideally there should be an upgrade script to change "<!-- -->"
into "<?pkidaemon ?>" in existing instances too.
This is a fair point; I've filed a ticket:
https://fedorahosted.org/pki/ticket/2234
2965
days inactive
2977
days old
3 comments
2 participants
participants (2)
-
Endi Sukma Dewata -
Fraser Tweedale