On Sat, Apr 12, 2008 at 8:21 PM, Kevin Quick <[EMAIL PROTECTED]> wrote: > > On 12 Apr 2008, at 1:08 AM, Michael Homer wrote: > > On Sat, Apr 12, 2008 at 7:16 PM, Kevin Quick <[EMAIL PROTECTED]> wrote: > >> > > > It's in the %decls (Perl) hash, typed to an array - cabal_variables > > and runhaskell should be listed there too. > > They are. Not in the diff you sent in the last message. The only RecipeLint changes are in the bash declarations array and to add `case "$recipe_type" in ...`. Anyway, I've made the changes locally myself > > Actually, what is cabal_variables intended for? It's merged with > > --prefix=, so should it really be cabal_options or is it used for > > variables in the same way as make_variables? It doesn't really make > > any difference to how they're entered or parsed, but we should make > > sure the name is right before we're committed to preserving > > compatibility with it. The use flag merging should happen with > > whatever the default, most common way of adding configuration options > > is. > > I was boilerplating from other recipes. I agree though, > cabal_options is more apropo. Do you want another patch, or do you > just want to do this yourself? I've also done this one. cabal_options and runhaskell.
Applied as r3181, with the changes mentioned above and a typo fix for `egrep -q` in MakeRecipe. -Michael _______________________________________________ gobolinux-devel mailing list gobolinux-devel@lists.gobolinux.org http://lists.gobolinux.org/mailman/listinfo/gobolinux-devel
