Thread: [Lxr-dev] [ lxr-Bugs-453387 ] variable 'v' range
Brought to you by:
ajlittoz
From: <no...@so...> - 2001-08-20 14:58:33
|
Bugs item #453387, was opened at 2001-08-20 07:58 You can respond by visiting: http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 Category: genxref Group: None Status: Open Resolution: None Priority: 5 Submitted By: Pavel Hlavnicka (pavel_hlavnicka) Assigned to: Nobody/Anonymous (nobody) Summary: variable 'v' range Initial Comment: I downloaded devel version 0.8 (no group available :) If you use the commented out example in the lxr.conf how to set the range of the variable 'v' it doesn't work. First, at the line 74 in genxref is this variable accesed directly, there should be @versions->varrange('v') I guess. Second, after hacking this line it still doesn't work, because the $LXR::Common::pathname (used in lxr.conf) is not set at the time of the execution. I've no clue, what was the original author's intention :) It seems, that the range sould be read once per processed file, because the releases (probably) and revisions (for sure) may differ for each file. But I didn't read the code so carefully to be sure. HTH ---------------------------------------------------------------------- You can respond by visiting: http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 |
From: <no...@so...> - 2001-09-26 09:34:16
|
Bugs item #453387, was opened at 2001-08-20 07:58 You can respond by visiting: http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 Category: genxref Group: None Status: Open Resolution: None Priority: 5 Submitted By: Pavel Hlavnicka (pavel_hlavnicka) Assigned to: Peder O. Klingenberg (pok) Summary: variable 'v' range Initial Comment: I downloaded devel version 0.8 (no group available :) If you use the commented out example in the lxr.conf how to set the range of the variable 'v' it doesn't work. First, at the line 74 in genxref is this variable accesed directly, there should be @versions->varrange('v') I guess. Second, after hacking this line it still doesn't work, because the $LXR::Common::pathname (used in lxr.conf) is not set at the time of the execution. I've no clue, what was the original author's intention :) It seems, that the range sould be read once per processed file, because the releases (probably) and revisions (for sure) may differ for each file. But I didn't read the code so carefully to be sure. HTH ---------------------------------------------------------------------- >Comment By: Malcolm Box (mbox) Date: 2001-09-26 02:33 Message: Logged In: YES user_id=215386 Is there any chance you could post the stuff you added to generic.conf to get php working? I'm trying to flesh the file out, since it currently doesn't support a lot of the languages that ctags can cope with. ---------------------------------------------------------------------- Comment By: Joseph Wilhelm (tarken) Date: 2001-09-17 13:48 Message: Logged In: YES user_id=326177 Hmm, on my last follow-up to this, perhaps it didn't work properly. I added a 'php' entry to generic.conf so that everything should work great in my PHP files... but I'm not getting any cross-references on functions or anything else. All I'm getting them on is require()'ed files in the current directory. Also, I looked at the lxr database in MySQL... and it's completely empty. Perhaps this is evidence that genxref didn't do quite what it was supposed to? :) ---------------------------------------------------------------------- Comment By: Joseph Wilhelm (tarken) Date: 2001-09-17 12:53 Message: Logged In: YES user_id=326177 Actually I got this fixed on my copy by making the line look like this: @versions = eval($config->{variables}{v}{range}); Instead of: @versions = @{$config->{variables}{v}{range}}; And it's working perfectly now :) ---------------------------------------------------------------------- Comment By: Peder O. Klingenberg (pok) Date: 2001-08-20 11:17 Message: Logged In: YES user_id=222352 See patch 453450. ---------------------------------------------------------------------- Comment By: Peder O. Klingenberg (pok) Date: 2001-08-20 10:59 Message: Logged In: YES user_id=222352 No, it doesn't work at the moment, because genxref is broken. I have fixed it, but the patch has not been committed yet. Your hunch is correct, it's intended to be called per file. Stay tuned. ---------------------------------------------------------------------- You can respond by visiting: http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 |
From: <no...@so...> - 2001-09-27 17:48:49
|
Bugs item #453387, was opened at 2001-08-20 07:58 You can respond by visiting: http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 Category: genxref Group: None Status: Open Resolution: None Priority: 5 Submitted By: Pavel Hlavnicka (pavel_hlavnicka) Assigned to: Peder O. Klingenberg (pok) Summary: variable 'v' range Initial Comment: I downloaded devel version 0.8 (no group available :) If you use the commented out example in the lxr.conf how to set the range of the variable 'v' it doesn't work. First, at the line 74 in genxref is this variable accesed directly, there should be @versions->varrange('v') I guess. Second, after hacking this line it still doesn't work, because the $LXR::Common::pathname (used in lxr.conf) is not set at the time of the execution. I've no clue, what was the original author's intention :) It seems, that the range sould be read once per processed file, because the releases (probably) and revisions (for sure) may differ for each file. But I didn't read the code so carefully to be sure. HTH ---------------------------------------------------------------------- Comment By: Joseph Wilhelm (tarken) Date: 2001-09-26 08:13 Message: Logged In: YES user_id=326177 Okay.. as far as I know, this is complete. I took the list of reserved words from the list on php.net. 'php' => { 'reserved' => ['and','$argv','$argc','break','case','class', 'continue','default','do','die','echo','else', 'elseif','empty','endfor','endforeach','endif', 'endswitch','endwhile','E_ALL','E_PARSE','E_ERROR', 'E_WARNING','exit','extends','FALSE','for','foreach ', 'function','HTTP_COOKIE_VARS','HTTP_GET_VARS', 'HTTP_POST_VARS','HTTP_POST_FILES','HTTP_ENV_VARS', 'HTTP_SERVER_VARS','if','global','list','new','not' , 'NULL','or','parent','PHP_OS','PHP_SELF','PHP_VERSI ON', 'print','return','static','switch','stdClass', 'this','TRUE','var','xor','virtual','while','__FILE __', '__LINE__','__sleep','__wakeup' ], 'spec' => ['comment', '/\*', '\*/', 'comment', '//', "\$", 'comment', '#', "\$", 'string', '"', '"', 'string', "'", "'", 'include', 'require', "\$", 'include', 'include', "\$", 'include', 'require_once', "\$", 'include', 'include_once', "\$" ], }, ---------------------------------------------------------------------- Comment By: Malcolm Box (mbox) Date: 2001-09-26 02:33 Message: Logged In: YES user_id=215386 Is there any chance you could post the stuff you added to generic.conf to get php working? I'm trying to flesh the file out, since it currently doesn't support a lot of the languages that ctags can cope with. ---------------------------------------------------------------------- Comment By: Joseph Wilhelm (tarken) Date: 2001-09-17 13:48 Message: Logged In: YES user_id=326177 Hmm, on my last follow-up to this, perhaps it didn't work properly. I added a 'php' entry to generic.conf so that everything should work great in my PHP files... but I'm not getting any cross-references on functions or anything else. All I'm getting them on is require()'ed files in the current directory. Also, I looked at the lxr database in MySQL... and it's completely empty. Perhaps this is evidence that genxref didn't do quite what it was supposed to? :) ---------------------------------------------------------------------- Comment By: Joseph Wilhelm (tarken) Date: 2001-09-17 12:53 Message: Logged In: YES user_id=326177 Actually I got this fixed on my copy by making the line look like this: @versions = eval($config->{variables}{v}{range}); Instead of: @versions = @{$config->{variables}{v}{range}}; And it's working perfectly now :) ---------------------------------------------------------------------- Comment By: Peder O. Klingenberg (pok) Date: 2001-08-20 11:17 Message: Logged In: YES user_id=222352 See patch 453450. ---------------------------------------------------------------------- Comment By: Peder O. Klingenberg (pok) Date: 2001-08-20 10:59 Message: Logged In: YES user_id=222352 No, it doesn't work at the moment, because genxref is broken. I have fixed it, but the patch has not been committed yet. Your hunch is correct, it's intended to be called per file. Stay tuned. ---------------------------------------------------------------------- You can respond by visiting: http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 |
From: <no...@so...> - 2001-10-24 15:33:02
|
Bugs item #453387, was opened at 2001-08-20 07:58 You can respond by visiting: http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 Category: genxref >Group: current cvs >Status: Closed >Resolution: Fixed Priority: 5 Submitted By: Pavel Hlavnicka (pavel_hlavnicka) Assigned to: Peder O. Klingenberg (pok) Summary: variable 'v' range Initial Comment: I downloaded devel version 0.8 (no group available :) If you use the commented out example in the lxr.conf how to set the range of the variable 'v' it doesn't work. First, at the line 74 in genxref is this variable accesed directly, there should be @versions->varrange('v') I guess. Second, after hacking this line it still doesn't work, because the $LXR::Common::pathname (used in lxr.conf) is not set at the time of the execution. I've no clue, what was the original author's intention :) It seems, that the range sould be read once per processed file, because the releases (probably) and revisions (for sure) may differ for each file. But I didn't read the code so carefully to be sure. HTH ---------------------------------------------------------------------- Comment By: Joseph Wilhelm (tarken) Date: 2001-09-26 08:13 Message: Logged In: YES user_id=326177 Okay.. as far as I know, this is complete. I took the list of reserved words from the list on php.net. 'php' => { 'reserved' => ['and','$argv','$argc','break','case','class', 'continue','default','do','die','echo','else', 'elseif','empty','endfor','endforeach','endif', 'endswitch','endwhile','E_ALL','E_PARSE','E_ERROR', 'E_WARNING','exit','extends','FALSE','for','foreach ', 'function','HTTP_COOKIE_VARS','HTTP_GET_VARS', 'HTTP_POST_VARS','HTTP_POST_FILES','HTTP_ENV_VARS', 'HTTP_SERVER_VARS','if','global','list','new','not' , 'NULL','or','parent','PHP_OS','PHP_SELF','PHP_VERSI ON', 'print','return','static','switch','stdClass', 'this','TRUE','var','xor','virtual','while','__FILE __', '__LINE__','__sleep','__wakeup' ], 'spec' => ['comment', '/\*', '\*/', 'comment', '//', "\$", 'comment', '#', "\$", 'string', '"', '"', 'string', "'", "'", 'include', 'require', "\$", 'include', 'include', "\$", 'include', 'require_once', "\$", 'include', 'include_once', "\$" ], }, ---------------------------------------------------------------------- Comment By: Malcolm Box (mbox) Date: 2001-09-26 02:33 Message: Logged In: YES user_id=215386 Is there any chance you could post the stuff you added to generic.conf to get php working? I'm trying to flesh the file out, since it currently doesn't support a lot of the languages that ctags can cope with. ---------------------------------------------------------------------- Comment By: Joseph Wilhelm (tarken) Date: 2001-09-17 13:48 Message: Logged In: YES user_id=326177 Hmm, on my last follow-up to this, perhaps it didn't work properly. I added a 'php' entry to generic.conf so that everything should work great in my PHP files... but I'm not getting any cross-references on functions or anything else. All I'm getting them on is require()'ed files in the current directory. Also, I looked at the lxr database in MySQL... and it's completely empty. Perhaps this is evidence that genxref didn't do quite what it was supposed to? :) ---------------------------------------------------------------------- Comment By: Joseph Wilhelm (tarken) Date: 2001-09-17 12:53 Message: Logged In: YES user_id=326177 Actually I got this fixed on my copy by making the line look like this: @versions = eval($config->{variables}{v}{range}); Instead of: @versions = @{$config->{variables}{v}{range}}; And it's working perfectly now :) ---------------------------------------------------------------------- Comment By: Peder O. Klingenberg (pok) Date: 2001-08-20 11:17 Message: Logged In: YES user_id=222352 See patch 453450. ---------------------------------------------------------------------- Comment By: Peder O. Klingenberg (pok) Date: 2001-08-20 10:59 Message: Logged In: YES user_id=222352 No, it doesn't work at the moment, because genxref is broken. I have fixed it, but the patch has not been committed yet. Your hunch is correct, it's intended to be called per file. Stay tuned. ---------------------------------------------------------------------- You can respond by visiting: http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 |
From: Jason D. S. <js...@de...> - 2001-08-20 17:25:47
|
no...@so... wrote: > > Bugs item #453387, was opened at 2001-08-20 07:58 > You can respond by visiting: > http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350 > > Category: genxref > Group: None > Status: Open > Resolution: None > Priority: 5 > Submitted By: Pavel Hlavnicka (pavel_hlavnicka) > Assigned to: Nobody/Anonymous (nobody) > Summary: variable 'v' range > > Initial Comment: > I downloaded devel version 0.8 (no group available :) > > If you use the commented out example in the lxr.conf > how to set the range of the variable 'v' it doesn't work. > > First, at the line 74 in genxref is this variable > accesed directly, there should be > @versions->varrange('v') I guess. > > Second, after hacking this line it still doesn't work, > because the $LXR::Common::pathname (used in lxr.conf) > is not set at the time of the execution. I've no clue, > what was the original author's intention :) > It seems, that the range sould be read once per > processed file, because the releases (probably) and > revisions (for sure) may differ for each file. But I > didn't read the code so carefully to be sure. Can LXR be set up to use a remote CVS repository as its source tree? It shouldn't be hard (althoug especially slow). If so, this would be a good way to set up the default configuration. jason |
From: <pe...@kl...> - 2001-08-20 18:04:10
|
Jason Dorje Short <js...@de...> writes: > Can LXR be set up to use a remote CVS repository as its source tree? It > shouldn't be hard (althoug especially slow). Not at the moment. It needs access to the cvs file tree. :( > If so, this would be a good way to set up the default configuration. I agree. Automatically indexing its own CVS tree by default, perhaps? But unfortunately vaporware at the moment. :( ...Peder... -- Cogito ergo panta rei. |
From: Malcolm B. <ma...@br...> - 2001-08-21 02:53:57
|
Peder O. Klingenberg wrote: >Jason Dorje Short <js...@de...> writes: > >>Can LXR be set up to use a remote CVS repository as its source tree? It >>shouldn't be hard (althoug especially slow). >> > >Not at the moment. It needs access to the cvs file tree. :( > This is something I would very much like to see, since if we are to support other SCMs we cannot rely on being able to directly access the underlying files. It seems to me that it would be possible to do this. CVS.pm needs access to the following info for a file: 1) The versions of that file 2) The contents of the file cvs status -v will give you all the tagged versions, which may be OK, and obviously cvs co gives the contents (as is currently done). As you say, this would be pretty slow without some aggressive caching, but it should at least work. >>If so, this would be a good way to set up the default configuration. >> > >I agree. Automatically indexing its own CVS tree by default, perhaps? >But unfortunately vaporware at the moment. :( > Indeed. Hmm, now, should I hack on this or on the CSS/HTML re-write :-) Malcolm |