search

iterate / mikrobloggeriet

1 stars 1 forks source link

mblog.sh: fix variant field uuid #103

Closed tingstad closed 5 months ago

tingstad commented 5 months ago

Jeg oppdaga at jeg ikke hadde lest reglene nøye nok. Man risikerer at en andel av genererte UUIDer blir feil.

https://en.wikipedia.org/wiki/Universally_unique_identifier#Version_4_(random)

Variant 1 is most common (bits 1 0).

teodorlu commented 5 months ago

Urelatert: det ser ut som mblog.sh krever at $EDITOR skal være ett program. Jeg har $EDITOR satt til emacsclient -c. For mblog tror jeg det funker -- da kjører jeg tokenization først, så jeg kjører emacsclient -c MINFIL i stedet for "emacsclient -c" MINFIL (et program som heter emacsclient -c med space i filnavn).

Jeg er også usikker på hva som faktisk er korrekt oppførsel her, hva man egentlig skal bruke $EDITOR til.

Men EDITOR="emacsclient -c" git commit gjør det den skal.

teodorlu commented 5 months ago

Jeg vil gjerne vite om vi har feilformaterte UUID-er på disk. Jeg lagde et issue: https://github.com/iterate/mikrobloggeriet/issues/104

tingstad commented 5 months ago

(Jeg merget siden jeg mener det er mer sannsynlig at den nye implementasjonen er riktig enn den gamle.)

teodorlu commented 5 months ago

Støttes!