#186 Generic.pm does unnecessary initialisation

current_cvs
closed-fixed
3
2011-12-21
2011-06-25
No

Initialisation of generic parser is don in sub read_config. Since it is called only once, it is a good place to proceed with various tests, such as tools version number. This is done with the Perl backtick construct, executing a shell command as an independent process.

Some security rules may forbid such a parallel execution from a web server (e.g. on SourceForge). Thus, initialisation aborts and LXR can't display files. To circumvent the problem, initialisation should detect whether it is call from shell contect (i.e. from genxref) or from web context.

Since there is no need of ctags in web context, this verification should not be attempted.

Discussion

  • Andre-Littoz

    Andre-Littoz - 2011-12-21

    All the tests on auxiliary tools have been moved to genxref which is easier since genxref is executed in shell context. Moreover, LXR data is always initialised by genxref which is run only once in a while, at least not every time you need LXR.

    Change in CVS for next release (0.10).

     
  • Andre-Littoz

    Andre-Littoz - 2011-12-21

    This has now been fixed in CVS.

    If you can install the new version and check that it solves your
    problem, then it would be very useful.

    Thanks for reporting this defect and helping to make LXR better.

     
  • Andre-Littoz

    Andre-Littoz - 2011-12-21
    • status: open --> open-accepted
     
  • Andre-Littoz

    Andre-Littoz - 2011-12-21
    • status: open-accepted --> closed-accepted
     
  • Andre-Littoz

    Andre-Littoz - 2011-12-21

    This has now been fixed in CVS.

    If you can install the new version and check that it solves your
    problem, then it would be very useful.

    Thanks for reporting this defect and helping to make LXR better.

     
  • Andre-Littoz

    Andre-Littoz - 2011-12-21
    • status: closed-accepted --> closed-fixed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks