From 0f0a89556565f7cf399e53da9366b90ef5d7a815 Mon Sep 17 00:00:00 2001 From: "bbaetz%cs.mcgill.ca" <> Date: Wed, 14 Nov 2001 23:12:23 +0000 Subject: Bug 107718 pt 2 - use subtraction instead of the ~ operator, which older mysql versions don't support. Remove the checksetup.pl restriction, too. r=jake x2 --- checksetup.pl | 10 +--------- process_bug.cgi | 7 +++++-- 2 files changed, 6 insertions(+), 11 deletions(-) diff --git a/checksetup.pl b/checksetup.pl index a72492226..0a3edac79 100755 --- a/checksetup.pl +++ b/checksetup.pl @@ -773,15 +773,7 @@ my $drh = DBI->install_driver($db_base) if ($my_db_check) { # Do we have the database itself? - # XXX - as part of the fix for dataloss bug 107718, we need the ~ - # operator, which isn't available in earlier versions, despite what - # the docs say. This is temporary, as we won't need bit fiddling - # once bug 60822 is fixed. - # A requirement for 3.23.x may become permenant though - see - # http://bugzilla.mozilla.org/show_bug.cgi?id=87958 - my $sql_want = "3.23.5"; - - #my $sql_want = "3.22.5"; # minimum version of MySQL + my $sql_want = "3.22.5"; # minimum version of MySQL # original DSN line was: # my $dsn = "DBI:$db_base:$my_db_name;$my_db_host;$my_db_port"; diff --git a/process_bug.cgi b/process_bug.cgi index 7f3d32098..9e90827f1 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -43,7 +43,8 @@ use vars %::versions, %::legal_platform, %::legal_priority, %::target_milestone, - %::legal_severity; + %::legal_severity, + %::superusergroupset; my $whoid = confirm_login(); @@ -563,7 +564,9 @@ if($::usergroupset ne '0') { } if ($groupAdd ne "0" || $groupDel ne "0") { DoComma(); - $::query .= "groupset = ((groupset & ~($groupDel)) | ($groupAdd))"; + # mysql < 3.23.5 doesn't support the ~ operator, even though + # the docs say that it does + $::query .= "groupset = ((groupset & ($::superusergroupset - ($groupDel))) | ($groupAdd))"; } } -- cgit v1.2.3-24-g4f1b