-
Andrei Pavel writes: I was trying to de-autoconfiscate Kea at https://gitlab.isc.org/isc-projects/kea, and ran into some obstacles. Some could be fixed or clarified. Hence this patch: more-extensions-in-deconfig-and-some-clarifications-in-howto.patch. Could not open MR directly. It was not clear to me that running makemake in the top-level directory means passing the top-level directory as a parameter. The script was stalling and I thought I had stumbled on a serious bug. I pondered fixing the script, but it turns out that it already tries to default to the current directory, but only after trying to accept input from stdin which is where my script was stuck at. I didn't understand the use of this stdin input, so I decided to suggest mentioning in the docs that you're supposed to run makemake .. We use raw string literals with custom delimiters, and those are not supported by unifdef. I fixed this by turning them into default delimiters in Kea. Added a paragraph to howto docs. Added .cc, .cc.in, .cc.skel, .h.in extensions to deconfig. They are all used in Kea. Also sorted extensions lexicographically if you don't mind. It's a good way to spot duplicates.
57d288beAndrei Pavel writes: I was trying to de-autoconfiscate Kea at https://gitlab.isc.org/isc-projects/kea, and ran into some obstacles. Some could be fixed or clarified. Hence this patch: more-extensions-in-deconfig-and-some-clarifications-in-howto.patch. Could not open MR directly. It was not clear to me that running makemake in the top-level directory means passing the top-level directory as a parameter. The script was stalling and I thought I had stumbled on a serious bug. I pondered fixing the script, but it turns out that it already tries to default to the current directory, but only after trying to accept input from stdin which is where my script was stuck at. I didn't understand the use of this stdin input, so I decided to suggest mentioning in the docs that you're supposed to run makemake .. We use raw string literals with custom delimiters, and those are not supported by unifdef. I fixed this by turning them into default delimiters in Kea. Added a paragraph to howto docs. Added .cc, .cc.in, .cc.skel, .h.in extensions to deconfig. They are all used in Kea. Also sorted extensions lexicographically if you don't mind. It's a good way to spot duplicates.
Loading