]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commitdiff
Merge branch 'master' of ssh://git.ikiwiki.info/srv/git/ikiwiki.info
authorJoey Hess <joey@gnu.kitenet.net>
Sat, 16 May 2009 20:42:47 +0000 (16:42 -0400)
committerJoey Hess <joey@gnu.kitenet.net>
Sat, 16 May 2009 20:42:47 +0000 (16:42 -0400)
1  2 
doc/todo/tracking_bugs_with_dependencies.mdwn

index ba653fdcc335fac66fe01ae21ec77d1eac698b86,b8c5b8f20157812de4c4dddf7de273693a81b429..3043cd9f25bc2dc81c65cd762ee9058d1bfc6ed1
@@@ -11,6 -11,8 +11,8 @@@ I like the idea of [[tips/integrated_is
  >
  >> I thought about this briefly, and got about that far.. glad you got
  >> further. :-) --[[Joey]]
+ >> Or, one could also refer to the language of [[!wikipedia description logics]]: their formulas actually define classes of objects through quantified relations to other classes. --Ivan Z.
  > 
  > Another option would be go with a more functional syntax.  The concept here would
  > be to allow a pagespec to appear in a 'pagespec function' anywhere a page can.  e.g.
  
  >>>> Yeah, guess that'd work. :-)
  
- > One quick further thought.  All the above discussion assumes that 'dependency' is the
+ > <a id="another_kind_of_links" />One quick further thought.  All the above discussion assumes that 'dependency' is the
  > same as 'links to', which is not really true.  For example, you'd like to be able to say
  > "This bug does not depend upon [ [ link to other bug ] ]" and not have a dependency.
  > Without having different types of links, I don't see how this would be possible.
  >
  > -- [[Will]]
  
+ >> I saw that this issue is targeted at by the work on [[structured page data#another_kind_of_links]]. --Ivan Z.
  Okie - I've had a quick attempt at this.  Initial patch attached.  This one doesn't quite work.
  And there is still a lot of debugging stuff in there.
  
@@@ -188,43 -192,6 +192,43 @@@ The following three inlines work for m
  I've lost track of the indent level, so I'm going back to not indented - I think this is a working [[patch]] taking into
  account all comments above (which doesn't mean it is above reproach :) ).  --[[Will]]
  
 +> Very belated code review of last version of the patch:
 +> 
 +> * `is_globlist` is no longer needed
 +> * `pagespec_translate` is already memoized, so the explicit call
 +>   to memoize when handling a define seems unnecessary?
 +> * I don't understand why the pagespec match regexp is changed
 +>   from having flags `igx` to `ixgs`. Don't see why you
 +>   want `.` to match '\n` in it, and don't see any `.` in the regexp 
 +>   anyway?
 +> * Some changes of `@_` to `%params` in `pagespec_makeperl` do not
 +>   make sense to me. I don't see where \%params is defined and populated,
 +>   except with `\$params{specFunc}`.
 +> * Seems that the only reason `match_glob` has to check for `~` is
 +>   because when a named spec appears in a pagespec, it is translated
 +>   to `match_glob("~foo")`. If, instead, `pagespec_makeperl` checked
 +>   for named specs, it could convert them into `check_named_spec("foo")`
 +>   and avoid that ugliness.
 +> * The changes to `match_link` seem either unecessary, or incomplete.
 +>   Shouldn't it check for named specs and call
 +>   `check_named_spec_existential`?
 +> * Generally, the need to modify `match_*` functions so that they
 +>   check for and handle named pagespecs seems suboptimal, if
 +>   only because there might be others people may want to use named
 +>   pagespecs with. It would be possible to move this check
 +>   to `pagespec_makeperl`, by having it check if the parameter
 +>   passed to a pagespec function looked like a named pagespec.
 +>   The only issue is that some pagespec functions take a parameter
 +>   that is not a page name at all, and it could be weird
 +>   if such a parameter were accidentially interpreted as a named
 +>   pagespec. (But, that seems unlikely to happen.)
 +> * I need to check if your trick to avoid infinite recursion
 +>   works if there are two named specs that recursively
 +>   call one-another. I suspect it does, but will test this
 +>   myself..
 +>  
 +> --[[Joey]] 
 +
  ----
  
      diff --git a/IkiWiki.pm b/IkiWiki.pm