File types not associated when using file_extensions on Windows #185
Comments
mrclary
added
the
type::bug
Note: this does not appear to establish file type association (conda/menuinst#185)
|
Hm, can you try with a leading period for the extensions? See example. "It works on CI", but CI runs as administrator by default, so we might need to adjust something. "Open With" has not been added per se, as it requires some extra registry keys. |
|
@mrclary did you have a chance to take a look at the workaround? |
|
@jaimergp, sorry, I have not yet had a chance to look at it. |
The short of itI suspect that an error occurs on install after the first item in the file extension list is handled resulting in no other extensions being handled, but the shortcut is produced. On uninstall, an error results due to the missing expected AssocFiles. The long of itSo I tried using This version of Spyder does not have file extensions listed in the However, I get the following errors after modifying spyder-menu.json
And attempting to uninstall:
If I only have one extension in the list, the install produces the same error, although the AssocFile is produced. The uninstall does not produce an error. So, I suspect that the error occurs on install after the first extension in the list is handled resulting in no other extensions being handled, but the shortcut is produced nonetheless. Then, on uninstall, the error is a result of the missing expected AssocFiles. |
|
Looks like the problem is at: menuinst/menuinst/platforms/win_utils/registry.py Lines 73 to 76 in db52885 So maybe it works if you remove the |
Note: this does not appear to establish file type association (conda/menuinst#185)
Note: this does not appear to establish file type association (conda/menuinst#185)
Note: this does not appear to establish file type association (conda/menuinst#185)
Note: this does not appear to establish file type association (conda/menuinst#185)
|
@jaimergp, with The only issue that remains is that the file type association in context menus appears as Python instead of Spyder (both name and icon). Nevertheless, associating with "Python" does, in fact, open the file in Spyder. Perhaps this is a result of the shortcut target using 
|
|
@jaimergp, actually there is still an issue with the registry when removing an environment. The registry key is removed, so perhaps it is trying to delete it again and it's no longer there? |
|
Also, FYI, |
Yes, this is a known issue: conda/conda#11092. I'll look into the other reports, thanks for giving it a try! Edit: Found these resources. Writing them down here for future reference:
|
|
Apparently Windows wants us to use "indirect strings" for those Progids attributes. We can't just put the text directly there. Instead, we are asked to bundle the custom strings in a resource binary ( Writing a PE file from scratch is technically possible with |
|
Huh, turns out at least one more person in the world has looked into this and reported that Windows 10 does support raw strings without the PE indirection. That simplifies things drastically. I'll open a PR once I can push to the repo. |
I'm wondering about this now. |
I think you are correct; it is exactly just a renamed launcher designed to run a file of its own name 
Indeed, doing as you suggest and replacing the shortcut target results in Spyder launching without a cmd window 😄. However, the Notably, |
Nope, this work-around does not work. Apparently, |
|
Pity, that's a current We could add the feature in conda-build + conda but only recent versions would be compatible and we are looking at a September release the earliest. |
|
Ah, it's already non-noarch. What if you copy # -*- coding: utf-8 -*-
import re
import sys
from YOUR.MODULE import YOUR_FUNCTION
if __name__ == '__main__':
sys.argv[0] = re.sub(r'(-script\.pyw?|\.exe)?$', '', sys.argv[0])
sys.exit(YOUR_FUNCTION()) |
🤔 |
|
Yes, my suggestion is to not generate the entry point in Windows and instead do it by hand in the build script. This way you get rid of the post-link / pre-unlink scripts. |
Note: this does not appear to establish file type association (conda/menuinst#185)
|
Thanks! |
Note: this does not appear to establish file type association (conda/menuinst#185)
Checklist
What happened?
@jaimergp, I get several errors when using
file_extensions. Theconstructor-based installer successfully installs the environment and shortcuts, however file types don't seem to be associated with Spyder, i.e. Spyder is not an available application to "open with" for expected file types.Additionally, attempting to uninstall or install with
menuinstresults in the following errors.Attempt to uninstall shortcut
Attempt to install shortcut
I get the same errors whether I run the installer as administrator or not, except the traceback file paths are
C:\ProgramData\spyder-6...instead. The first error appears to result because thespyder-6.AssocFilecregistry key was not created as expected. I'm puzzled why the second error occurs even when run from a cmd window with administrator privileges: why wasn't aPermissionErrorraised whenmenuinstwas run from the installer?.Conda Info
Conda Config
Conda list
Additional Context
spyder-menu.json
{ "$schema": "https://json-schema.org/draft-07/schema", "$id": "https://schemas.conda.io/menuinst-1.schema.json", "menu_name": "{{ DISTRIBUTION_NAME }} spyder", "menu_items": [ { "name": "Spyder 6", "description": "Scientific PYthon Development EnviRonment", "icon": "{{ MENU_DIR }}/spyder.{{ ICON_EXT }}", "activate": false, "terminal": false, "command": [""], "platforms": { "win": { "desktop": true, "app_user_model_id": "spyder.Spyder", "command": ["{{ PREFIX }}/pythonw.exe", "{{ PREFIX }}/Scripts/spyder-script.py"], "file_extensions": [ "bat", "c", "cc", "cfg", "cl", "cmd", "cpp", "css", "cxx", "desktop", "diff", "enaml", "f", "f03", "f08", "f2k", "f77", "f90", "f95", "for", "h", "h", "hh", "hpp", "htm", "html", "hxx", "inf", "ini", "ipy", "ipynb", "jl", "js", "json", "m", "md", "nsh", "nsi", "patch", "po", "pot", "pro", "properties", "pxd", "pxi", "py", "pyw", "pyx", "reg", "rej", "rst", "scss", "session", "txt", "xml", "yaml", "yml" ] }, "linux": { "Categories": [ "Development", "Science" ], "command": ["{{ PREFIX }}/bin/spyder", "$@"], "StartupWMClass": "Spyder", "MimeType": [ "text/x-script.python", "text/plain", "text/html", "text/xml", "text/x-c", "text/x-perl" ] }, "osx": { "precommand": "pushd \"$(dirname \"$0\")\" &>/dev/null", "command": ["./python", "{{ PREFIX }}/bin/spyder", "$@"], "link_in_bundle": { "{{ PREFIX }}/bin/python": "{{ MENU_ITEM_LOCATION }}/Contents/MacOS/python" }, "CFBundleName": "Spyder 6", "CFBundleIdentifier": "org.spyder-ide.Spyder", "CFBundleVersion": "6.0.0a4.dev152", "CFBundleDocumentTypes": [ { "CFBundleTypeName": "text document", "CFBundleTypeRole": "Editor", "LSHandlerRank": "Default", "CFBundleTypeIconFile": "spyder.icns", "LSItemContentTypes": [ "com.apple.applescript.text", "com.apple.ascii-property-list", "com.apple.audio-unit-preset", "com.apple.binary-property-list", "com.apple.configprofile", "com.apple.crashreport", "com.apple.dashcode.css", "com.apple.dashcode.javascript", "com.apple.dashcode.json", "com.apple.dashcode.manifest", "com.apple.dt.document.ascii-property-list", "com.apple.dt.document.script-suite-property-list", "com.apple.dt.document.script-terminology-property-list", "com.apple.property-list", "com.apple.rez-source", "com.apple.scripting-definition", "com.apple.structured-text", "com.apple.traditional-mac-plain-text", "com.apple.xcode.ada-source", "com.apple.xcode.apinotes", "com.apple.xcode.bash-script", "com.apple.xcode.configsettings", "com.apple.xcode.csh-script", "com.apple.xcode.entitlements-property-list", "com.apple.xcode.fortran-source", "com.apple.xcode.glsl-source", "com.apple.xcode.ksh-script", "com.apple.xcode.lex-source", "com.apple.xcode.make-script", "com.apple.xcode.mig-source", "com.apple.xcode.pascal-source", "com.apple.xcode.strings-text", "com.apple.xcode.tcsh-script", "com.apple.xcode.yacc-source", "com.apple.xcode.zsh-script", "com.apple.xml-property-list", "com.netscape.javascript-source", "com.scenarist.closed-caption", "com.sun.java-source", "com.sun.java-web-start", "net.daringfireball.markdown", "org.khronos.glsl-source", "org.oasis-open.xliff", "public.ada-source", "public.assembly-source", "public.bash-script", "public.c-header", "public.c-plus-plus-header", "public.c-plus-plus-source", "public.c-source", "public.case-insensitive-text", "public.comma-separated-values-text", "public.csh-script", "public.css", "public.delimited-values-text", "public.dylan-source", "public.filename-extension", "public.fortran-77-source", "public.fortran-90-source", "public.fortran-95-source", "public.fortran-source", "public.html", "public.json", "public.ksh-script", "public.lex-source", "public.log", "public.m3u-playlist", "public.make-source", "public.mig-source", "public.mime-type", "public.module-map", "public.nasm-assembly-source", "public.objective-c-plus-plus-source", "public.objective-c-source", "public.opencl-source", "public.pascal-source", "public.patch-file", "public.perl-script", "public.php-script", "public.plain-text", "public.python-script", "public.rss", "public.ruby-script", "public.script", "public.shell-script", "public.source-code", "public.tcsh-script", "public.text", "public.utf16-external-plain-text", "public.utf16-plain-text", "public.utf8-plain-text", "public.utf8-tab-separated-values-text", "public.xhtml", "public.xml", "public.yacc-source", "public.yaml", "public.zsh-script" ] } ] } } } ] }The text was updated successfully, but these errors were encountered: