#438: Allow a configuration.ccl for the flesh -----------------------+---------------------------------------------------- Reporter: eschnett | Owner: hinder Type: defect | Status: review Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: -----------------------+---------------------------------------------------- Changes (by hinder):

* owner: => hinder

#438: Allow a configuration.ccl for the flesh -----------------------+---------------------------------------------------- Reporter: eschnett | Owner: hinder Type: defect | Status: review Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: -----------------------+----------------------------------------------------

Comment (by eschnett):

Does this mean that you approve of the idea of having an empty configuration.ccl in the flesh?

I can't produce a patch to add the empty configuration.ccl file; this isn't possible with svn -- you probably mean to ensure that such an empty file is committed if the patch is applied.

A thorn called "Cactus" is how the flesh is represented internally. The flesh is treated specially in many ways, and this particular loop can't handle the flesh because it determines arrangement and thorn name from a directory name, which doesn't make sense for the flesh. The original if statement only skips two lines, while the next statement also skips the remainder of the loop for the flesh, which needs to be skipped for the same reason as above.

The change to ConfigurationParser.pl changes debug code that is commented out by default. It could go into a separate patch; however, given the complexity of preparing, submitting, reviewing, and then applying patches, this doesn't seem worthwhile. I'll do it if you think this helps.

The change to CreateConfigurationBindings corrects an inconsistency; it ensures that each requirement is processed only once.

The latter two changes can be applied independently of allowing a configuration.ccl for the flesh, but I found them necessary during testing, so I would call them a prerequisite, although I can't recall the exact circumstances under which these changes were required to correct a problem.

#438: Allow a configuration.ccl for the flesh -----------------------+---------------------------------------------------- Reporter: eschnett | Owner: hinder Type: defect | Status: review Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: -----------------------+----------------------------------------------------

Comment (by hinder):

I didn't know that the SVN patch format doesn't support adding empty files. If there is a good reason why it's hard to make the code work without having an empty configuration.ccl file in the flesh, then I'd say this was a minor problem and perfectly fine to apply. If on the other hand it is easy to correct, then by all means do so :)

#438: Allow a configuration.ccl for the flesh -----------------------+---------------------------------------------------- Reporter: eschnett | Owner: hinder Type: defect | Status: review Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: -----------------------+----------------------------------------------------

Comment (by eschnett):

It is somewhat difficult to allow no configuration.ccl file in the flesh. One needs to write a makefile rule that detects if a configuration.ccl is there and wasn't before, or if it is there and has changed, or if it is not there and more but used to be there, but doesn't complain if it is missing. Makefiles aren't designed for this; if a dependency is missing, this usually leads to an error.

Cactus introduces a special logic for this, explicitly keeping track of which thorns have a configuration.ccl file and which don't, and since the flesh and the thorns are treated somewhat differently we would have to duplicate this logic for the flesh. I don't think it's worth the effort.