#991638 nodejs: Please enable build on 32-bit PowerPC (powerpc)

Package:
src:nodejs
Source:
nodejs
Submitter:
John Paul Adrian Glaubitz
Date:
2021-10-19 23:45:04 UTC
Severity:
normal
Tags:
#991638#5
Date:
2021-07-29 12:39:06 UTC
From:
To:
Hi!

I just noticed today that NodeJS has principal support for 32-bit PowerPC
which it internally calls it "ppc". While the build does not succeed at
first attempt (see below), the fact that NodeJS supports 32-bit PowerPC
in general means that we should be able to get it working.

Since "powerpc" is not a release architecture, it's perfectly fine if the
package fails to build from source as this will not inhibit the propagation
of the nodejs package to testing.

Thus, could you please modify debian/control and debian/rules accordingly
such that nodejs can be built on 32-bit PowerPC (powerpc)? See attached
patch.

===========================================================================

Current build failure of NodeJS version 14 on 32-bit PowerPC:

In file included from ../deps/v8/src/objects/visitors.h:9,
                 from ../deps/v8/src/heap/heap.h:33,
                 from ../deps/v8/src/heap/factory.h:16,
                 from ../deps/v8/src/execution/isolate.h:28,
                 from ../deps/v8/src/api/api.h:10,
                 from ../deps/v8/src/api/api-arguments.h:8,
                 from ../deps/v8/src/api/api-arguments.cc:5:
../deps/v8/src/objects/code.h:439:2: error: #error Unknown architecture.
  439 | #error Unknown architecture.
      |  ^~~~~
In file included from ../deps/v8/src/execution/isolate.h:18,
                 from ../deps/v8/src/api/api.h:10,
                 from ../deps/v8/src/api/api-arguments.h:8,
                 from ../deps/v8/src/api/api-arguments.cc:5:
../deps/v8/src/objects/code.h:441:55: error: 'kHeaderPaddingSize' was not declared in this scope
  441 |   STATIC_ASSERT(FIELD_SIZE(kOptionalPaddingOffset) == kHeaderPaddingSize);
      |                                                       ^~~~~~~~~~~~~~~~~~
../deps/v8/src/base/macros.h:200:43: note: in definition of macro 'STATIC_ASSERT'
  200 | #define STATIC_ASSERT(test) static_assert(test, #test)
      |                                           ^~~~

Thanks,
Adrian
#991638#10
Date:
2021-07-29 19:22:11 UTC
From:
To:
Hi,


Le jeu. 29 juil. 2021 à 14:51, John Paul Adrian Glaubitz <
glaubitz@physik.fu-berlin.de> a écrit :

Thanks, it's worth a shot.
There is a chance nodejs won't work on all ppc 32bit cpu supported by
debian, though.
Let's see.

Jérémy
#991638#13
Date:
2021-07-29 19:32:24 UTC
From:
To:
Hello,

Bug #991638 in nodejs reported by you has been fixed in the
Git repository and is awaiting an upload. You can see the commit
message below and you can check the diff of the fix at:

https://salsa.debian.org/js-team/nodejs/-/commit/0f10129d20ea6f0f573fc2b25ac414b267972cbd

(this message was generated automatically)
-- 
Greetings

https://bugs.debian.org/991638
#991638#20
Date:
2021-07-29 20:23:38 UTC
From:
To:
Hi Jeremy!

It should work on the standard 32-bit PowerPC baseline which includes
AltiVec. Other systems such as PowerPC E500 had their own Debian port
anyway (powerpcspe).
NodeJS will definitely need some work on 32-bit PowerPC to get it working,
but enabling the build in the first place should be the first step.

FWIW, there is also a bootstrapping problem with NodeJS as it requires
node-arcon for building which in turn depends on nodejs itself.

So, we'll need to add an sbuild profile which allows to build the nodejs
package without to build the initial binary package for a bootstrap.

I'll file a separate bug for that.

Adrian
#991638#25
Date:
2021-08-04 15:02:46 UTC
From:
To:
32 bit powerpc does not require altivec.  e300 runs it fine and sure
doesn't have altivec.  G3 powermac had no altivec either.  A few packages
(like vlc for example) are broken and assume altivec is always present.

powerpcspe was for the offshoot that didn't use the normal FPU (had a
signal processor instruction set instead) like the e500 (but not e500mc).
#991638#28
Date:
2021-10-19 23:34:45 UTC
From:
To:
Hello,

Bug #991638 in nodejs reported by you has been fixed in the
Git repository and is awaiting an upload. You can see the commit
message below and you can check the diff of the fix at:

https://salsa.debian.org/js-team/nodejs/-/commit/0f10129d20ea6f0f573fc2b25ac414b267972cbd

(this message was generated automatically)
-- 
Greetings

https://bugs.debian.org/991638
#991638#31
Date:
2021-10-19 23:43:39 UTC
From:
To:
Hello,

Bug #991638 in nodejs reported by you has been fixed in the
Git repository and is awaiting an upload. You can see the commit
message below and you can check the diff of the fix at:

https://salsa.debian.org/js-team/nodejs/-/commit/dc627a50458f3619d7a4e2d21349fc7b273c412a

(this message was generated automatically)
-- 
Greetings

https://bugs.debian.org/991638