Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build fail (Arch) #1341

Open
gobborg opened this issue Jan 29, 2022 · 9 comments
Open

Build fail (Arch) #1341

gobborg opened this issue Jan 29, 2022 · 9 comments

Comments

@gobborg
Copy link

gobborg commented Jan 29, 2022

(1) Issue/Bug Description:
error making: gnome-shell-extension-pop-shell
When attempting to install the shell, everything worked until the build. The error is src/settings.ts:25:24 - error TS2345: Argument of type 'unknown is not assignable to parameter of type 'string'.
25 global.log(err).

Here is a screenshot:
image

(2) Steps to reproduce (if you know):
Try to install via yay -S gnome-shell-extension-pop-shell and choose default installation options.

(3) Expected behavior:
Successful build

(4) Distribution (run cat /etc/os-release):
Arch Linux

(5) Gnome Shell version:
41.2 (Yes, I know this is not 3.36.)

(6) Pop Shell version (run apt policy pop-shell or provide the latest commit if building locally):
aur/gnome-shell-extension-pop-shell-bin 1.2.0-4

(7) Where was Pop Shell installed from:
yay -S gnome-shell-extension-pop-shell

(8) Monitor Setup (2 x 1080p, 4K, Primary(Horizontal), Secondary(Vertical), etc):
Primary Horizontal 16:9

(9) Other Installed/Enabled Extensions:
aur/pop-shell-shortcuts-git 1:r30.52cc83f-2

(10) Other Notes:
I was able to install and build with the precompiled yay -S gnome-shell-extension-pop-shell-bin.

@0n0w1c
Copy link

0n0w1c commented Jan 31, 2022

I ran into the same compile error attempting to build it from the AUR for aarch64.

@0n0w1c
Copy link

0n0w1c commented Feb 1, 2022

The -git package in the AUR builds properly, and it builds for aarch64 too.

@kn0wmad
Copy link

kn0wmad commented Feb 3, 2022

Same error building from AUR, aarch64, Gnome 41.3. I know this is not listed as a supported version, but was hoping to give this a try.

@colbymorrison
Copy link

yay -S gnome-shell-extension-pop-shell-git worked for me. GNOME 41.3.

@c-b5r
Copy link

c-b5r commented Mar 5, 2022

System:

  • x86_64
  • gnome 41.3

I am having trouble building gnome-shell-extension-pop-shell-git for over a month now (stripped down to the essential kinds of errors):
src/extension.ts:30:15 - error TS2305: Module '"events"' has no exported member 'ExtEvent'.
src/extension.ts:46:9 - error TS2339: Property 'GlobalEvent' does not exist on type 'typeof EventEmitter'.
../../../../../node_modules/@types/node/globals.global.d.ts:1:13 - error TS2451: Cannot redeclare block-scoped variable 'global'.
../../../../../node_modules/@types/node/timers.d.ts:22:35 - error TS2339: Property 'setTimeout' does not exist on type 'Global'.

Considering @colbymorrison is able to build successfully:
Might this be an issue with node somehow? Maybe the (module) version dependencies are not resolved correctly or something... It looks to me as if my node version is somehow newer, thus now throws errors of incorrect/incompatible "old" code... Unfortunately, I have no expertise when it comes to node, so my guess could be completely wrong...

@jixbo
Copy link

jixbo commented Apr 30, 2022

I was facing the same issue as @CharlieBra7o.
There was a node_modules in my home folder, which was being used for the install.
Deleting the node_modules folder rm ~/node_modules/ fixed the issue for me using ubuntu 22.04.

BTW I had to build jammy_master branch, to get the extension to work in ubuntu 22.04

@c-b5r
Copy link

c-b5r commented May 9, 2022

@jixbo : Thank you for the tip with the user node_modules directory! After removing/renaming it I was also able to build master_jammy for gnome-shell 42.0-1. As far as I can see everything is working perfectly!

@picnoir
Copy link

picnoir commented Oct 17, 2022

Same error on NixOS here on the latest release here.

It seems like it has been fixed on master by 8be10fa.

This commit is not included in the latest release. It might make sense to either backport it, either cutting a new release containing it.

@chatodux
Copy link

chatodux commented Dec 6, 2022

I'm facing the same issue, in my case using RHEL 9.1. Typescript version 4.8.4.

gitgud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants