Skip to content

Build mdadm with mdcheck tool #784

Merged
merged 7 commits into from
May 23, 2018
Merged

Build mdadm with mdcheck tool #784

merged 7 commits into from
May 23, 2018

Conversation

donald
Copy link
Collaborator

@donald donald commented May 23, 2018

We need to scrub our raids. Fastest way to get there seems to be to use the mdcheck tools provided by mdadm. This is a rebuild of mdadm which adds usr/bin/mdcheck.

Because I was touching this, I fixed the gcc 7.3 build problems and removed the udev rule files by the way.

gcc 7.3.0 has some more warnings, which are triggered by the source. Do
not make these fatal.
The target 'everything' does not include mdadm which is required by make
install, just some variants like mdadm.static or mdadm.O2. We don't want
compilation done in the install phase.

The default target includes 'mdadm' but not all things we want to install.

Explicitly enumerate the build targets we want.
Testing showed, that a parallel build works. Remove option.
Install the mdcheck script provided by this package. It can be used to
scrub the md raids.
We don't want raid assembly done by udev, because we use some mariux
specific tool for it. By not installing these udev rules, we no longer
need to mask them which is currently done by the mdadm-fixup package.

Remove udev rule files from image.
This package is no longer required, because the package mdadm has been
modified to not install the rule files. Remove this package.
@pmenzel
Copy link
Collaborator

pmenzel commented May 23, 2018

The Makefile has:

UDEVDIR := $(shell $(PKG_CONFIG) --variable=udevdir udev 2>/dev/null)
ifndef UDEVDIR
 UDEVDIR = /lib/udev
endif

MarIuX givs:

$ pkg-config --variable=udevdir udev
/lib/udev

@pmenzel pmenzel changed the title Rebuild mdadm Build mdadm with mdcheck tool May 23, 2018
@pmenzel pmenzel merged commit 8659cfe into master May 23, 2018
@donald donald deleted the rebuild-mdadm branch September 14, 2018 08:46
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants