Ticket #10690 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Build packages in RelWithDbgInfo by default

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Harry Jeffery

Description

For nightly and unstable packages it is useful to have debugging symbols within the libraries. For release packages it will be set to Release.

Change History

comment:1 Changed 6 years ago by Martyn Gigg

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Martyn Gigg

  • Status changed from assigned to inprogress

Use RelWithDbgInfo as default build type for build servers

Refs #10690

Changeset: c1ffd510bb2b54d67c66d0bd3095c12745fb619f

comment:3 Changed 6 years ago by Martyn Gigg

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

This is being verified as pull request #84.

comment:4 Changed 6 years ago by Harry Jeffery

  • Status changed from verify to verifying
  • Tester set to Harry Jeffery

comment:5 Changed 6 years ago by Pete Peterson

retest this please

comment:6 Changed 6 years ago by Ross Whitfield

retest this please

comment:7 Changed 6 years ago by Harry Jeffery

  • Status changed from verifying to closed

Merge pull request #84 from mantidproject/feature/10690_relwithdbg_package_builds

Use RelWithDbgInfo as default build type for build servers

Full changeset: 19a1a866e74f630aaa0b28bc529f371e75c32963

comment:8 Changed 6 years ago by Martyn Gigg

@eXeC64: Any reason this was merged before the builds marked it okay?

comment:9 Changed 6 years ago by Harry Jeffery

@martyngigg: I reviewed the changes, and the logs from the builds, and http://builds.mantidproject.org/job/pull_requests/173/ showed the OS X, rhel6, and Ubuntu builds to be well under way before they were aborted (due to the rhel6 machine running out of disk space). Given the nature of the changes, if they had been breaking, the build wouldn't have gotten off the ground.

Combined with the ticket itself saying that "Code review should be sufficient here", I felt I'd seen sufficient evidence that the changes were correct.

comment:10 Changed 6 years ago by Martyn Gigg

Okay. For future issues it would be good to put a comment in to that effect when merging. Thanks

comment:11 Changed 6 years ago by Harry Jeffery

Okay, I'll be sure to do that in future.

comment:12 Changed 6 years ago by Pete Peterson

This has ballooned all of the libraries. I'm suspicious it is because we are over-linking/exporting things. I'm reverting and reopening.

comment:13 Changed 6 years ago by Pete Peterson

  • Status changed from closed to verify

This is being verified as pull request #88.

comment:14 Changed 6 years ago by Pete Peterson

@martyngigg This changes the .so from ~10M to ~100M. We need to figure out how to reduce our exported symbols first.

comment:15 Changed 6 years ago by Pete Peterson

  • Status changed from verify to closed

Merge pull request #88 from mantidproject/revert-84-feature/10690_relwithdbg_package_builds

Revert "Use RelWithDbgInfo as default build type for build servers"

Full changeset: da9b5042ba5ff7140b113d53934c66745ed2613a

comment:16 Changed 6 years ago by Martyn Gigg

Okay. Something to revisit after we sort out the linking mess.

On Fri, 5 Dec 2014 18:30 Pete Peterson <notifications@…> wrote:

This has ballooned all of the libraries. I'm suspicious it is because we are over-linking/exporting things. I'm reverting and reopening.

— Reply to this email directly or view it on GitHub <https://github.com/mantidproject/mantid/pull/84#issuecomment-65833027>.

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11532

Note: See TracTickets for help on using tickets.