autobuild3 icon indicating copy to clipboard operation
autobuild3 copied to clipboard

PM: Non-MPMs should also get a chance to load so of their lib helper funcs.

Open MingcongBai opened this issue 10 years ago • 7 comments

rpmarch is still absent from autobuild3.

MingcongBai avatar Aug 14 '15 19:08 MingcongBai

This doesn't break anything now, I am rephrasing the issue.

Artoria2e5 avatar Aug 14 '15 20:08 Artoria2e5

@Arthur2e5 This breaks rpm installation, as it looks for rpm file from armhf (first term possible?).

MingcongBai avatar Aug 14 '15 20:08 MingcongBai

@MingcongBai As far as I can see, ARCH_RPM[armel]=armfhp.

Artoria2e5 avatar Aug 14 '15 20:08 Artoria2e5

@Arthur2e5 But it happened on my x86_64 machine.

MingcongBai avatar Aug 14 '15 20:08 MingcongBai

ARCH_RPM[amd64]=x86_64, so look elsewhere for the bug. e.g. How the hell we called it.

Artoria2e5 avatar Aug 14 '15 20:08 Artoria2e5

We only have two references to rpmarch, both running in almost the same way, which falls from CROSS to ARCH. So how could this happen?

# rpm/install
rpm $PM_ROOTPARAM -Uvh /root/rpmbuild/RPMS/$(rpmarch ${CROSS:-$ARCH})/$PKGNAME-$PKGVER-$PKGREL*.rpm
# rpm/pack
rpmbuild $([ "$CROSS" ] && rpmarch $CROSS) -bb abspec || aberr "RPM PACKING FAILED."

Artoria2e5 avatar Aug 14 '15 20:08 Artoria2e5

@Arthur2e5 So how about this?

abtrycmd rpmarch || rpmarch(){ echo "${ARCH_RPM[$1]:-$1}"; }

It would not make sense not to include this in pack as well, eh?

MingcongBai avatar Aug 14 '15 20:08 MingcongBai