[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside
From: |
Stefan Monnier |
Subject: |
bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside |
Date: |
Sat, 11 May 2019 08:54:47 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) |
>> Instead, you should state the property you want, and then let each
>> file-name-handler decide how to handle that property. E.g.:
>>
>> (let ((write-region-provides-raw-file-contents t))
>> (write-region ...))
>>
>> Of course, the problem here is that by using dynamic scoping, this var
>> will affect all write-regions that will take place during this call,
>> which might lead to more problems.
>
> I see, I hadn't realized that tar-mode still used dynamic scoping (I
> imagine converting it to lexical is a lot of work).
All files use dynamically scoped variables. `lexical-binding` only
determines the scoping to use for those vars that aren't declared as
dynamically scoped. The fact that tar-mode.el hasn't (yet) been
converted to use lexical-binding has no effect on the above example
(because I presume there that write-region-provides-raw-file-contents
would be a variable declared somewhere in files.el as being dynamically
scoped, like file-name-handler-alist).
>> Maybe it would be better to have it as an argument to `write-region`, or
>> to introduce a new function?
>>
>> Or maybe, not use `write-region` but `copy-file` instead (which already
>> provides the expected semantics (i.e. it doesn't (un)compress or
>> (en/de)crypt, IIUC).
>
> It looks like `copy-file` only deals with files and not buffers, so we
> would need to find a way to use it from tar-mode.
I know. Maybe we can extend it to allow the source to be a buffer?
> Looking at the source in fileio.c, it also seems like it uses
> `find-file-name-handler` for `FILE` or `NEWNAME`, so it's possible it
> won't be useful for us.
Not sure why you think that could make it not useful.
> If we added a new argument to `write-region` like you mentioned, say
> `RAW`, could we then use symbol properties to decide whether the found
> file name handler can be used or not?
No, no: we do want the file-name-handler to be called.
We just want it to receive enough info to determine how it will do its
job (e.g. whether it needs to compress/encrypt the data or not).
Stefan
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Federico Tedin, 2019/05/08
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Stefan Monnier, 2019/05/08
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Federico Tedin, 2019/05/09
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Stefan Monnier, 2019/05/09
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Federico Tedin, 2019/05/10
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Stefan Monnier, 2019/05/10
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Federico Tedin, 2019/05/10
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Stefan Monnier, 2019/05/10
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Federico Tedin, 2019/05/10
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside,
Stefan Monnier <=
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Federico Tedin, 2019/05/11
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Eli Zaretskii, 2019/05/11
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Stefan Monnier, 2019/05/11
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Eli Zaretskii, 2019/05/11
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Stefan Monnier, 2019/05/11
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Eli Zaretskii, 2019/05/11
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Stefan Monnier, 2019/05/11
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Eli Zaretskii, 2019/05/12
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Stefan Monnier, 2019/05/12
- bug#35495: 27.0.50; Untarring an archive with a keyring.gpg inside, Eli Zaretskii, 2019/05/12