-
selsta
TheCharlatan: depends/native_ds_store.mk and depends/native_mac_alias.mk seem to have stopped working due to the repo being down, is this something you can look into?
-
selsta
I guess I can fix it myself by looking at the bitcoin repo
-
xmr-pr
selsta opened pull request #6771: depends: fix broken links for ds_store / mac_alias
-
xmr-pr
-
xiphon
selsta: fluffypony: "hmmmmmm getting a "Error: internal error: Transaction sanity check failed" on a transfer" -> wallet code does 3 attempts to fetch new outputs and re-run tx sanity check again. Tx sanity check routine logs a error message with the distinct reason on failure, please enable "verify" log category (or adjust log_level)
-
fluffypony
ah - I thought it was a known issue and did the xfer with a couple of smaller amounts
-
fluffypony
will see if I can repro with the change still in the wallet
-
kayront
-
xmr-pr
kayront opened issue #6773: Feature Request: Optionally Categorize Adressbook Entries
-
xmr-pr
-
xmr-pr
kayront opened issue #6772: Feature Request: Addressbook import/export
-
xmr-pr
-
fluffypony
ok cool I can reproduce it still
-
fluffypony
getting a new log now
-
fluffypony
sanitised log outoput
-
fluffypony
-
fluffypony
cc xiphon
-
Snipa
.merges
-
xmr-pr
6600 6693
-
hyc
we really need to quit calling our releases e.g. "Windows 64-bit" and "Mac OS-X 64-bit"
-
hyc
they should be "Windows x86-64" and "Mac OS-X x86-64"
-
hyc
there are Windows releases on ARM
-
hyc
and there will soon be Mac OSX releases on ARM too
-
selsta
hyc: mac releases will be one binary for both platforms
-
selsta
but for Windows I agree
-
hyc
oh, Mac will keep using fat binaries huh, ok
-
moneromooo
I binary that runs on x86_64 and ARM ?
-
hyc
yeah
-
hyc
same as when they transitioned from M68K to PPC, and PPC to x86
-
hyc
fat binaries, contains objet code for multiple platforms
-
moneromooo
Like, two actual binaries in a shell ?
-
hyc
yep
-
hyc
Mac files are structured objects, with multiple tagged streams
-
moneromooo
It seemed preposterous, but thinking about it, it's just #!/bin.sh exec $0`-name -m`...
-
moneromooo
"$@"
-
hyc
not a shell, it's their binary file format
-
hyc
actually, it's their *file* format, period. any file can have multiple forks inside
-
selsta
macos does not support 32 bit anymore so "macOS" might be enough in the future
-
moneromooo
Data files which give you different data depending on the arch you're on ?
-
hyc
the runtime loader knows which fork it wants to find, ignores the rest
-
moneromooo
Malware gonna love that.
-
selsta
afaik it even strips the unneeded part on first startup
-
hyc
no, you can see all the forks
-
moneromooo
Oh OK.
-
hyc
but it lets you tag them so you can keep things separate. and there are standard fork IDs like 'RSRC' for graphical resources, etc. Stuff that flat filesystems had to keep in a separate .RSC file
-
selsta
monero-project/monero #6771 needs a review so that we can fix depends / travis CI
-
vtnerd
macOS could release llvm bitcode bianries that finish ASM translation at runtime. the appstore has done this behind the scenes for a while, but dont know if they are bold enough to do it on desktop
-
dEBRUYNE
Is that Github link supposed to error out?
-
dEBRUYNE
^ selsta
-
vtnerd
the link works for me
-
vtnerd
unless there was another one since 20:05, got disconnected for a bit
-
selsta
nothing errors out here
-
dEBRUYNE
I get 'page not found' for the /archive/ stuff
-
dEBRUYNE
selsta, vtnerd: I meant the Github link in the commit, sorry
-
selsta
dEBRUYNE: you need to merge the link together manually
-
iDunk
Maybe you forgot to add 1.1.0.tar.gz
-
iDunk
Or whatever it was.
-
selsta
yes
-
dEBRUYNE
Ah
-
dEBRUYNE
Will see myself out :p
-
selsta
-
dEBRUYNE
ty
-
selsta
Travis CI passed so the fix should be correct
-
moneromooo
Anyone got the old tarballs to compare ? Changing URL and hash at the same time without the old tarballs to compare is wide open to abuse.
-
selsta
the old repo is gone
-
selsta
I don’t have a copy
-
selsta
maybe someone who has gitian setup can look if a copy is still somewhere
-
selsta
FWIW bitcoin switched to the github repo over a year ago
-
selsta
-
selsta
found it in gitian caches
-
selsta
but maybe someone else wants to confirm this as I opened the PR?
-
selsta
the diff is due to a python3 patch getting applied
-
moneromooo
Patch seems good.
-
selsta
ok seems like the majority of CLSAG code is testing related, this shouldn’t be too difficult to review