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

innosetup logs Permission Denied, but still succeeds #3339

Closed
MCOfficer opened this issue Apr 19, 2019 · 13 comments · Fixed by #3399
Closed

innosetup logs Permission Denied, but still succeeds #3339

MCOfficer opened this issue Apr 19, 2019 · 13 comments · Fixed by #3399

Comments

@MCOfficer
Copy link
Contributor

MCOfficer commented Apr 19, 2019

While trying to create a manifest for Celestia, i ran into this weird error:

Installing 'celestia' (1.6.1) [64bit]
Loading celestia-161.exe from cache
Checking hash of celestia-161.exe ... ok.
Unpacking innosetup... Move-Item : Der Zugriff auf den Pfad "C:\Users\Florian\scoop\apps\celestia\1.6.1\_scoop_unpack\{app}\locale" wurde verweigert.
In C:\Users\Florian\scoop\apps\scoop\current\lib\decompress.ps1:64 Zeichen:51
+ ... ildItem "$dir\_scoop_unpack\{app}" -r | Move-Item -dest "$dir" -force
+                                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : WriteError: (C:\Users\Floria...ck\{app}\locale:DirectoryInfo) [Move-Item], IOException
+ FullyQualifiedErrorId : MoveDirectoryItemIOError,Microsoft.PowerShell.Commands.MoveItemCommand
done.
Linking ~\scoop\apps\celestia\current => ~\scoop\apps\celestia\1.6.1
Creating shim for 'celestia'.
'celestia' (1.6.1) was installed successfully!

(That German part essentially translates to "Permission denied").

Despite this, the installation succeeds and as far as i can tell, no part of the locale folder is missing - however, all its contents are part of the app directory (...\scoop\apps\celestia\current\), the locale folder doesn't exist.

You can find the manifest version here, let me know if you need any more info.

@r15ch13
Copy link
Member

r15ch13 commented Apr 20, 2019

Hm, works for me:

image

Regex tweak: celestia-win-[^>]+>Celestia\s+([\d.]+)

@MCOfficer
Copy link
Contributor Author

MCOfficer commented Apr 20, 2019

Odd. I can reproduce it on my system, even after clearing the cache. I also made sure the celestia folder has been properly removed before reinstalling, i have no idea why it's not working correctly.

i should mention that, due to a fuckup with locate, i can't use scoop install from within the powershell, i use it from cmd instead. to me, that seems more like a screwed environment, that's why i didn't report it.

Also, my PS version:

λ» $PSVersionTable.PSVersion

Major  Minor  Build  Revision
-----  -----  -----  --------
5      1      17134  590

@r15ch13
Copy link
Member

r15ch13 commented Apr 20, 2019

cmd works:
image

due to a fuckup with locate, i can't use scoop install from within the powershell, i use it from cmd instead.

How did this happen? 😄

@MCOfficer
Copy link
Contributor Author

MCOfficer commented Apr 21, 2019

it's not offtopic at all... but whatever:
i tried to install an mlocate port for ps, it failed gloriously and it took the ps noob that i am 2 hours to remove it. and, apparently, i didn't remove it cleanly, for scoop fails on scoop update X (not just scoop update) and scoop install:

λ» scoop install vim
locate : Die Benennung "local:locate" wurde nicht als Name eines Cmdlet, einer Funktion, einer Skriptdatei oder eines ausführbaren Programms erkannt. Überprüfen Sie die Schreibweise des Namens, oder ob der Pfad korrekt ist (sofern enthalten), und wiederholen Sie den Vorgang.
In C:\Users\Florian\scoop\apps\scoop\current\lib\depends.ps1:25 Zeichen:38
+     $null, $manifest, $null, $null = locate $app $bucket
+                                      ~~~~~~
    + CategoryInfo          : ObjectNotFound: (local:locate:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Couldn't find manifest for 'vim'.

that aside, since this bug seems to affect only me, should i PR celestia to scoop-extras?

@r15ch13
Copy link
Member

r15ch13 commented Apr 21, 2019

locate is a bad function name anyway. Will change it to Find-Manifest in a fix later.
Yes, please add a PR for celestia 😄

@MCOfficer
Copy link
Contributor Author

MCOfficer commented Apr 23, 2019

It happened with another app, stellarium, and this time it had consequences.

λ» scoop install stellarium
Installing 'stellarium' (0.19.0) [64bit]
Loading stellarium-0.19.0.1-win64.exe from cache
Checking hash of stellarium-0.19.0.1-win64.exe ... ok.
Unpacking innosetup... Move-Item : Der Zugriff auf den Pfad "C:\Users\Florian\scoop\apps\stellarium\0.19.0\_scoop_unpack\{app}\textures" wurde verweigert.
In C:\Users\Florian\scoop\apps\scoop\current\lib\decompress.ps1:64 Zeichen:51
+ ... ildItem "$dir\_scoop_unpack\{app}" -r | Move-Item -dest "$dir" -force
+                                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (C:\Users\Floria...\{app}\textures:DirectoryInfo) [Move-Item], IOException
    + FullyQualifiedErrorId : MoveDirectoryItemIOError,Microsoft.PowerShell.Commands.MoveItemCommand

done.
Linking ~\scoop\apps\stellarium\current => ~\scoop\apps\stellarium\0.19.0
Creating shim for 'stellarium'.
Creating shortcut for Stellarium (stellarium.exe)
'stellarium' (0.19.0) was installed successfully!

Apparently the textures are required - so much that it refuses to launch, without any error.

While looking for a logfile, i found a innounp.log, which might be useful.

Edit: here's the manifest in question.

@niheaven
Copy link
Member

niheaven commented Apr 25, 2019

It's a failure caused by unpack_inno, Move-Item cannot handle too many files, and will be fixed in #3204...

@MCOfficer
Copy link
Contributor Author

MCOfficer commented Apr 28, 2019

umm... i can't confirm, because now something different happens.

~
λ» scoop install pipp celestia
Couldn't find manifest for 'innounp'.

~
λ» scoop info innounp
Could not find manifest for 'innounp'.

~
λ» scoop install innounp
WARN  'innounp' (0.48) is already installed.
Use 'scoop update innounp' to install a new version.


~
λ» scoop update innounp
innounp: 0.48 (latest version)
Latest versions for all apps are installed! For more information try 'scoop status'

@Ash258
Copy link
Contributor

Ash258 commented Apr 28, 2019

Run scoop udpate again

@MCOfficer
Copy link
Contributor Author

wow, bucket switch. did i update right in the middle of that, or do you always have to update twice?

anyways,

~
λ» scoop install pipp celestia
WARN  'celestia' (1.6.1) is already installed. Skipping.
Installing 'pipp' (2.5.9) [64bit]
Loading pipp_install_x64_2.5.9.zip from cache
Checking hash of pipp_install_x64_2.5.9.zip ... ok.
Extracting pipp_install_x64_2.5.9.zip ... Failed to extract files from C:\Users\Florian\scoop\apps\pipp\2.5.9\pipp_install_x64_2.5.9.zip.
Log file:
  ~\scoop\apps\pipp\2.5.9\innounp.log

innounp.log

@Ash258
Copy link
Contributor

Ash258 commented Apr 28, 2019

It is zip file. I don't think innosetups are boundled as zips.

@Ash258
Copy link
Contributor

Ash258 commented Apr 28, 2019

Yeah.

A

You need to extract that zip file first and then do something like installer.script = "Expand-Inno file $dir -Removal"

@MCOfficer
Copy link
Contributor Author

uuh. you're right, although that poses like a dozen new questions for me.

but as far as this issue is concerned, celestia and stellarium install as intended. sorry for panicking.

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

Successfully merging a pull request may close this issue.

4 participants