-
Notifications
You must be signed in to change notification settings - Fork 367
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
blackbox_update_all_files fails with unknown reason #249
Comments
Is there a chance that blackbox-files.txt was rewritten using DOS-style line endings? https://github.com/StackExchange/blackbox#protect-the-line-endings |
I have checked both files the The version of
$ echo $BASH_VERSION
3.2.57(1)-release |
That's interesting. Do other blackbox_* commands work? Does "make test" run to completion? (you'll need to move your ~/.gnupg directory out of the way temporarily) Could you add this debugging line and show me the output?
|
Thanks @TomOnTime @tlimoncelli for helping out. It looks like the root cause was the MacOS shell behaviour (tested in iTerm at least) which converts directories starting with capital letter to lower case. See below difference in the output the directory name
It used to work before but after the latest upgrade to Mac OS Sierra ver. 10.13.4 it stopped working. |
Previous osx file systems were case preserving, but insensitive. Is it
possible your recent upgrade also changed your filesystem?
https://www.quora.com/Why-does-OS-X-choose-to-have-a-case-insensitive-file-system-instead-of-a-case-sensitive-one
…On Thu, Apr 19, 2018, 7:50 AM Constantin Bugneac ***@***.***> wrote:
Thanks @TomOnTime <https://github.com/TomOnTime> @tlimoncelli
<https://github.com/tlimoncelli> for helping out.
It looks like the root cause was the MacOS shell behaviour (tested in
iTerm at least) which converts directories starting with capital letter to
lower case. See below difference in the output the directory name Git in
ERROR vs git in PWD:
ERROR: service/creds.json not found in /Users/user/Git/secrets/keyrings/live/blackbox-files.txt
PWD=/Users/user/git/infra-jobs-secrets
It used to work before but after the latest upgrade to Mac OS Sierra ver.
10.13.4 it stopped working.
I don't know what went wrong but renaming directory from Git to git
sorted the issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#249 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA-zpsGPE2nAbD1p3gRxjz5RQZ_Iut-7ks5tqHoDgaJpZM4TYpdw>
.
|
I don't think the filesystem has changed during upgrade. Here is what I've got now:
Looks like blackbox bash script(s) have some portions which differently treat directories with capital letters thus the issue. |
I'd like to fix this... or at least have the error message give a hint about what to do. Sadly I haven't been able to reproduce the problem. I've tried a number of things but no luck. If I had to guess, the problem is in vcs_relative_path() |
Having the same error message in Ubuntu 16.04 running in a VM. Managed to solve the problem when I realized the project folder was sym-linked elsewhere. I just had to run it in the original path. |
The same issue on Windows, tried to add file from windows. I tried to decrypt file, but got the same issue
|
@VZhuck Are you using Cygwin or MinGW? |
Hi @tlimoncelli, Looks like root cause:
However when I run the same commands under Linux Subsystem on Windows (Ubuntu), it works as expected |
That makes sense. The system was designed for a filesystem that is case sensitive. If a directory is named "foo" and you get there by "cd FOO", all bets are off. |
When I run
blackbox_update_all_files
I get this meaningless error with doesn't provide a clue on what the problem could be. It's on latest Mac OS.But when I cat
blackbox-files.txt
I can see thereservice/creds.json
listed.Any idea what could be wrong ?
The text was updated successfully, but these errors were encountered: