==================================================================
  Please DO NOT REPLY to this mail or send email to the developers
  about this bug. Please follow-up to Bugzilla using this 
link:http://bugs.contribs.org/show_bug.cgi?id=8293

  Have you checked the Frequently Asked Questions (FAQ)?
    http://wiki.contribs.org/SME_Server:Documentation:FAQ

  Please also take the time to read the following useful guide:
    http://www.chiark.greenend.org.uk/~sgtatham/bugs.html
==================================================================


            Bug ID: 8293
           Summary: smeserver-affa does not write rpms-missing.txt
    Classification: Contribs
           Product: SME Contribs
           Version: 8.1
          Hardware: ---
                OS: ---
            Status: CONFIRMED
          Severity: normal
          Priority: P3
         Component: smeserver-affa (unmaintained)
          Assignee: [email protected]
          Reporter: [email protected]
        QA Contact: [email protected]

Created attachment 4559
  --> http://bugs.contribs.org/attachment.cgi?id=4559&action=edit
rpms-missing.txt diff

I noticed that affa was not writing the rpms-missing.txt file again on either
v8 or 8.1

I attach a diff file to fix this plus a spelling mistake and a removal of a
tab. I'd do a proper spec file but I'm knackered, and I'm not sure if what I
have done is correct

I'm don't understand why this failed - explanations welcome !

I checked the variables and in each case they were :

$job{'SMEServer'} = yes
$job{'RPMCheck'} = yes
$job{'remoteHostName'} = myserver.domain.co.uk

Original line:
return if $job{'SMEServer'} eq 'no' or $job{'RPMCheck'} ne 'yes';

Changed to:
return if $job{'SMEServer'} eq 'no' or $job{'RPMCheck'} eq 'no';

Original line:
return if $job{'SMEServer'} eq 'no' or $job{'remoteHostName'} eq 'localhost' or
$job{'RPMCheck'} ne 'yes';

Changed to:
return if $job{'SMEServer'} eq 'no' or $job{'remoteHostName'} eq 'localhost' or
$job{'RPMCheck'} eq 'no';


So changing 'ne' to 'eq' fixed it, but I don't understand why because 'ne' look
like it should work.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
_______________________________________________
Mail for each SME Contribs bug report
To unsubscribe, e-mail [email protected]
Searchable archive at http://lists.contribs.org/mailman/public/contribteam/

Reply via email to