On 2015-04-10 15:15, [email protected] wrote: > From: Riku Voipio <[email protected]> > > create_package() function tries to resolve used architecture > for everry package. Split the setting the architecture to a > new function, set_debarch(), called once on startup. > > This allows using debarch from other parts of script as > needed. > --- > scripts/package/builddeb | 14 ++++++++++---- > 1 file changed, 10 insertions(+), 4 deletions(-) > > diff --git a/scripts/package/builddeb b/scripts/package/builddeb > index a9858a4..e397815 100755 > --- a/scripts/package/builddeb > +++ b/scripts/package/builddeb > @@ -27,8 +27,15 @@ create_package() { > chown -R root:root "$pdir" > chmod -R go-w "$pdir" > > + # Create the package > + dpkg-gencontrol $forcearch -Vkernel:debarch="${debarch:-$(dpkg > --print-architecture)}" -p$pname -P"$pdir" > + dpkg --build "$pdir" .. > +} > + > +set_debarch() { > # Attempt to find the correct Debian architecture > - local forcearch="" debarch="" > + forcearch="" > + debarch="" > case "$UTS_MACHINE" in > i386|ia64|alpha) > debarch="$UTS_MACHINE" ;; > @@ -73,10 +80,8 @@ create_package() { > if [ -n "$debarch" ] ; then > forcearch="-DArchitecture=$debarch" > fi > + export forcearch debarch
You do not need to export the variables. But it will improve the readability of the script if you initialize the variables in the toplevel scope. Also, the fallback for empty $debarch is $(dpkg --print-architecture). It makes sense to set it once in the new function. Michal -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected] Archive: https://lists.debian.org/[email protected]
