search

BrightonBTC / bija

MIT License
67 stars 16 forks source link

Setup fails #81

Closed yomajo closed 1 year ago

yomajo commented 1 year ago 
$ git clone --recurse-submodules https://github.com/BrightonBTC/bija
Cloning into 'bija'...
remote: Enumerating objects: 2961, done.
remote: Counting objects: 100% (611/611), done.
remote: Compressing objects: 100% (245/245), done.
remote: Total 2961 (delta 367), reused 453 (delta 366), pack-reused 2350
Receiving objects: 100% (2961/2961), 809.89 KiB | 2.80 MiB/s, done.
Resolving deltas: 100% (2222/2222), done.
Submodule 'python-nostr' (https://github.com/BrightonBTC/python-nostr) registered for path 'python_nostr'
Cloning into '<my_dir>/bija/python_nostr'...
remote: Enumerating objects: 271, done.        
remote: Counting objects: 100% (121/121), done.        
remote: Compressing objects: 100% (39/39), done.        
remote: Total 271 (delta 96), reused 84 (delta 82), pack-reused 150        
Receiving objects: 100% (271/271), 43.06 KiB | 1.30 MiB/s, done.
Resolving deltas: 100% (161/161), done.
fatal: remote error: upload-pack: not our ref c3fd8ac414b21d0e16afbc8cd48d8a2ea8b47037
fatal: Fetched in submodule path 'python_nostr', but it did not contain c3fd8ac414b21d0e16afbc8cd48d8a2ea8b47037. Direct fetching of that commit failed.
fatal:

Subsequent docker compose up:

...
=> CACHED [4/8] COPY requirements.txt ./                                  0.0s
 => CACHED [5/8] RUN pip install --no-cache-dir -r requirements.txt        0.0s
 => ERROR [6/8] COPY python_nostr/requirements.txt ./python_nostr/require  0.0s
------
 > [6/8] COPY python_nostr/requirements.txt ./python_nostr/requirements.txt:
------
failed to solve: failed to compute cache key: "/python_nostr/requirements.txt" not found: not found
yomajo commented 1 year ago

Sure this is to be blamed. Dead link as of writing: https://github.com/BrightonBTC/python-nostr/tree/c3fd8ac414b21d0e16afbc8cd48d8a2ea8b47037

BrightonBTC commented 1 year ago

Hmm, yeah, that's definitely the issue. I'm looking in to what's happened here. Cheers.

BrightonBTC commented 1 year ago

I've just merged a whole load of changes in to the main branch that fix this, in fact it removes the submodule altogether. Wasn't quite ready to do so TBH so not sure how stable it is but figured it's better than leaving the main branch broken/unusable.