Ticket #8323 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Latest Win 7 package triggers Sophos malware warning about Uninstall.exe

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: blocker Milestone: Release 3.0
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description


Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Martyn Gigg

Get any master, clean or incremental, package from jenkins and try and install it. As the installation finishes Sophos pops up warning that MantidInstall\Uninstall.exe contains malware.

This must be a false positive but we need to track down what is causing it.

comment:3 Changed 7 years ago by Martyn Gigg

Rename VISION IDF to avoid triggering Sophos malware warning.

When building a Windows package Sophos claims that the NSIS-generated Uninstall.exe contains malware. This must be a false positive and it has been submitted to Sophos for testing.

We really need to be able produce working kits for the release and the rename seems to avoid the problem. Refs #8323

Changeset: 9fb929fdfc7c786abba1d45fb9f0c3ea39b4e59c

comment:4 Changed 7 years ago by Martyn Gigg

  • Status changed from inprogress to verify
  • Resolution set to fixed

Branch: bugfix/8323_sophos_warning

Tester: For some reason this does not show up with the develop package but does with all of the current packages. A proper test would be to merge to master, build and then generate the package locally. This will require a complete build of Mantid, including VATES.

I think a brief check of the rename should suffice though as nothing will actually be relying on the filename.

comment:5 Changed 7 years ago by Martyn Gigg

Note: I have also submitted the offending file to Sophos for inspection.

comment:6 Changed 7 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:7 Changed 7 years ago by Nick Draper

Alternate testing approach.

Merge, let the build servers build and test the resulting package.

We would always want to test the final package anyway.

comment:8 Changed 7 years ago by Nick Draper

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/8323_sophos_warning'

Full changeset: 6c3a3941e9ae44dfbbfebc33467df605ad8d8625

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9168

Note: See TracTickets for help on using tickets.