search

iterate / mikrobloggeriet

0 stars 0 forks source link

`mblog config repo-path $PWD` feiler på Mikrobloggeriet versjon 6d5ff3, Babashka versjon v1.3.176) #53

Closed teodorlu closed 7 months ago

teodorlu commented 7 months ago

Rapportert av @tingstad.

Observert oppførsel

Fra Slack: https://garasjen.slack.com/archives/C05MH5RCLH3/p1701942474928919

10:45 :) richard@mac ~/dev/mikrobloggeriet* master                                                                                                
$ mblog config repo-path $PWD                                                                                                                     

----- Error --------------------------------------------------------------------                                                                  
Type:     clojure.lang.ArityException                                                                                                             
Message:  Wrong number of args (1) passed to: babashka.fs/xdg-config-home                                                                         
Location: /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:17:12                                                                    

----- Context ------------------------------------------------------------------                                                                  
13: ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;                                                              
14: ;; Low-level helpers for managing the config file                                                                                             
15:                                                                                                                                               
16: (defn- config-file []                                                                                                                         
17:   (fs/file (fs/xdg-config-home "mikrobloggeriet") "config.edn"))                                                                              
               ^--- Wrong number of args (1) passed to: babashka.fs/xdg-config-home                                                               
18:                                                                                                                                               
19: (defn- load-config []                                                                                                                         
20:   (into (sorted-map)                                                                                                                          
21:         (if (fs/exists? (config-file))                                                                                                        
22:           (edn/read-string (slurp (config-file)))                                                                                             

----- Stack trace --------------------------------------------------------------                               
mikrobloggeriet.cli                      - /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:17:12                                   
mikrobloggeriet.cli/config-file          - /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:17:3                                    
mikrobloggeriet.cli/config-file          - /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:16:1                                    
mikrobloggeriet.cli                      - /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:21:25                                   
mikrobloggeriet.cli                      - /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:21:13                                   
... (run with --debug to see elided elements)                                                                                                     
mikrobloggeriet.cli/config-set-repo-path - /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:62:1  
mikrobloggeriet.cli                      - /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:120:22
mikrobloggeriet.cli                      - /Users/richard/dev/mikrobloggeriet/src/mikrobloggeriet/cli.clj:333:3
clojure.core/apply                       - <built-in> 
user                                     - <expr>:1:44

Research

(@teodorlu)

1-arity-signaturen til babashka.fs/xdg-config-home kom i babashka/fs v0.4.18 (2023-05-11).

Babashka-versjonen til Richard er babashka v1.3.176, fra 2023-03-18.

Ser ut som vi kun støtter nye Babashka-versjoner, uten at vi har dokumentert det noe sted.

Løsningsforslag A

Sjekk om brukeren kjører en støttet versjon av Babashka i CLI-et, avbryt hvis vi kjører en versjon som ikke er støttet.

Løsningsforslag B

Skriv om koden til å støtte gamle Babashka-versjoner.

teodorlu commented 7 months ago

Jeg gikk for

Løsningsforslag B Skriv om koden til å støtte gamle Babashka-versjoner.

Don't break userspace! Prodsatt per https://github.com/iterate/mikrobloggeriet/commit/dcf1fd0bb99f464d319570980927f68b1432e53e, og Richard sier at "nå gikk det bedre".