Ticket #6103 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Modyfy CMAKE files to remove annoying message about dublicated integer definitions

Reported by: Alex Buts Owned by: Alex Buts
Priority: major Milestone: Release 2.4
Component: Mantid Keywords:
Cc: taylorrj@… Blocked By:
Blocking: Tester: Roman Tolchenov

Description (last modified by Alex Buts) (diff)

On windows system very often complains about multiple definitions of defined value integers e.g. uint32_t, int64_t etc...

This is caused by incomplete/incorrect definitions of stdint.h, cstdint.hpp and internal Matntid variables visibility. It can be tuned in studio by proper defining include and HAVE_STDINT_H or BOOST_CSTDINT_HPP preprocessor keys in studio, but in Mantid studio files are build by CMAKE.

I do not much understands how the stuff works , but it annoys me most while developing on Windows.

Would be asking Russell to have a look here too.

Change History

comment:1 Changed 8 years ago by Alex Buts

  • Description modified (diff)

comment:2 Changed 8 years ago by Alex Buts

refs #6103 remove main part of annoying messages

but not all of them.

Changeset: 6cb41db909ea624f605f0a983034578fb1e6e6b0

comment:3 Changed 8 years ago by Alex Buts

  • Owner changed from Alex Buts to Russell Taylor
  • Status changed from new to assigned

comment:4 Changed 8 years ago by Alex Buts

refs #6103 Fixing unix warning

these warnings have nothing to do with the ticket but the ticket caused large rebuild which let these warnings happens.

Changeset: 071c164219f67ce23230cb56635b5cd8d24177a8

comment:5 Changed 8 years ago by Alex Buts

refs #6103 remove main part of annoying messages

but not all of them.

Changeset: 6cb41db909ea624f605f0a983034578fb1e6e6b0

comment:6 Changed 8 years ago by Alex Buts

refs #6103 Fixing unix warning

these warnings have nothing to do with the ticket but the ticket caused large rebuild which let these warnings happens.

Changeset: 071c164219f67ce23230cb56635b5cd8d24177a8

comment:7 Changed 8 years ago by Alex Buts

  • Owner changed from Russell Taylor to Alex Buts

comment:8 Changed 8 years ago by Alex Buts

  • Status changed from assigned to accepted

comment:9 Changed 8 years ago by Alex Buts

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

Either due to this, or due to other changes, the message does not appear any more.

Tester should fully recompile mantid on Windows and should not see these messages.

comment:10 Changed 8 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:11 Changed 8 years ago by Roman Tolchenov

Didn't see any warning related to stdint.

comment:12 Changed 8 years ago by Roman Tolchenov

  • Status changed from verifying to closed

No, I saw some but not a lot.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6949

Note: See TracTickets for help on using tickets.