#155 genxref --reindexall with MySQL causes unknown table error

closed-out-of-date
Andre-Littoz
5
2011-03-26
2010-02-12
Patrick Gartung
No

Running genxref --reindexall when using MySQL 4.x produces an unkonown table error from mysql server.

Using the syntax suggested at this site
http://www.portfolioofpb.com/blog/mysql-sql-error-unknown-table-in-MULTI+DELETE-solved

I modified lib/LXR/Index/Mysql.pm to use the correct syntax for mysql 4.x

Discussion

  •  
    Attachments
  • Andre-Littoz
    Andre-Littoz
    2011-03-13

    MySQL 4.x ended 31 December 2010. Consequently is there still a need for including this correction in the main line?

    If somebody does need it, please resubmit and I'll create a (short lived) branch in the CVS.

     
  • Andre-Littoz
    Andre-Littoz
    2011-03-13

    • status: open --> pending-out-of-date
     
  • Malcolm Box
    Malcolm Box
    2011-03-14

    Maintaining compatibility with MySQL 4.x is useful as there are plenty of places still running this.

    Does the proposed change work on 5.x - if so it seems it would be worth keeping this in the main trunk.

     
  • Andre-Littoz
    Andre-Littoz
    2011-03-17

    • status: pending-out-of-date --> open
     
  • Andre-Littoz
    Andre-Littoz
    2011-03-17

    OK, I'll check to see if proposed change is "universal".
    If not, since this part of LXR is rather object oriented, I'll create another DB backend which could be selected through the 'dbname' config parameter: using :mysql4: instead of plain :mysql:

     
  • Andre-Littoz
    Andre-Littoz
    2011-03-25

    • status: open --> pending-remind
     
  • Andre-Littoz
    Andre-Littoz
    2011-03-25

    • assigned_to: nobody --> ajlittoz
     
  • Andre-Littoz
    Andre-Littoz
    2011-03-26

    Closed with the agreement of submitter:
    Was probably submitter against release 0.9.5 meanwhile Mysql.pm underwent many changes among:
    - no more aliases in SQL queries (which was causing the syntax error),
    - change of 'release' now a reserved word in MySQL 5.x to 'releaseid'
    - many small changes in sorting results
    In release 0.9.8 Mysql.pm is deemed to be compatible with MySQL 4.x and 5.x

     
  • Andre-Littoz
    Andre-Littoz
    2011-03-26

    • status: pending-remind --> closed-out-of-date