612564 – sys-cluster/mpich-3.2-r1 fails to build - rm: cannot remove ‘/var/tmp/portage/sys-cluster/mpich-3.2-r1/image/usr/include/mpif*’: No such file or directory

Bug 612564 - sys-cluster/mpich-3.2-r1 fails to build - rm: cannot remove ‘/var/tmp/portage/sys-cluster/mpich-3.2-r1/image/usr/include/mpif*’: No such file or directory

Summary: sys-cluster/mpich-3.2-r1 fails to build - rm: cannot remove ‘/var/tmp/portage...

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Justin Bronder (RETIRED)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2017-03-13 23:14 UTC by soya
Modified:	2017-04-12 20:45 UTC (History)
CC List:	3 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
emerge partial mpi.log (mpi.log,781.36 KB, text/x-log) 2017-03-13 23:14 UTC, soya	Details
Fixed logic that would previously allow rm to fail & die (mpich-3.2-r1.ebuild.patch,1.54 KB, patch) 2017-04-12 17:20 UTC, Michael Gilroy	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description soya 2017-03-13 23:14:36 UTC

Created attachment 466986 [details]
emerge partial mpi.log

sys-cluster/mpich-3.2-r1 fails to emerge

Comment 1 Justin Bronder (RETIRED) gentoo-dev

2017-03-14 15:01:45 UTC

Can you paste the output of 'emerge --info' please?

Comment 2 Michael Gilroy 2017-04-12 17:20:27 UTC

Created attachment 469884 [details, diff]
Fixed logic that would previously allow rm to fail & die

Reproducing the error:
- disable fortran (USES="-fortran")
- enable multilib ABI (ABI_X86="64 32")

Changed the multilib_src_install() so that the rm should not fail. Previously it was trying to remove files that were never generated in the case that it was not using fortran and multilib builds were enabled.

Comment 3 Justin Bronder (RETIRED) gentoo-dev

2017-04-12 20:45:05 UTC

I mucked with the comment lines a little (the unconditional install one from the old ebuild was confusing to me) and merged.

Thanks!


commit 18ac817afbb862936e4c4aa3a46d15b7eddd9dae (HEAD -> master, origin/master, origin/HEAD)
Author: Michael Gilroy <michael.gilroy24@gmail.com>
Date:   Wed Apr 12 15:14:23 2017 +0000

    sys-cluster/mpich: Fixed multilib_src_install() logic

    Previously in the case where compiles used -fortran and
    multilib_is_native_abi was true, the rm would fail and the build would
    die. The nested conditional prevents this.

    Bug: #612564