- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Mar 30 2020
Mar 29 2020
I made some effort to find an approach here, but I think it needs to wait until Toolsets define our config/caching better. In particular, guaranteeing this naively in arc requires running a subprocess (to execute locale -a, to enumerate available locales on the system) and possibly functionally-testing hundreds of locales (or, in the general case, an arbitrarily enormous number of locales). The functional test for UTF8 behavior requires running another subprocess, to echo 🐑.
Mar 26 2020
Mar 25 2020
use repository api
If that doesn't immediately work, yell at me and I can grab this and figure out what's up.
(Actual change makes perfect sense.)
Try something like this:
Mar 22 2020
D21044 may resolve this alone, but I suspect there will be at least a little bit of followup work so I'm going to leave this open for the moment.
Mar 20 2020
(Also, hello after all this time! Hope you're all doing well over at phacility.)
- For completeness, explicitly handle "Undefined offset" (raised for undefined numeric offsets) in addition to "Undefined index" (raised for undefined string indexes).
For my own internal reference, PHI1234 has related discussion.
Thanks! I added you to Blessed Committers, so you should be able to land this. See that project description for help if you run into issues, or you can likely use "Land Revision" from the web UI here, or I can land it for you if all of that is more trouble than it's worth.
- code review
Thanks for the code review and archeology! I did some archeology on our end and indeed, it was removed on Sep 26, 2018 when we forked this diff workflow for exactly the reason in this diff. With the --no-verify the LFS objects were not being pushed to our staging repo, so our Jenkins jobs wouldn't have them and would fail. So I do think this the right fix :)
- oops wrong way
Nice find! I see there is already a --no-verify in this upstream version but we didn't have it in ours... maybe because of the lfs version? So maybe this diff is actually improving the life of lfs folks?
- go with --no-verify
Instead of GIT_LFS_SKIP_SMUDGE=1, try git push --no-verify -- <remote-uri> ...? That just skips the pre-push hook, which seems to work locally:
- Revert "only run fetch if git for-each-ref is empty"
- Revert "fix spelling"
- Revert "handle the case where you need to remove it before adding it"
- Revert "Use a named remote and branches for staging to help git-lfs"
- try the double push thing for lfs repos
As much as possible I'd like arc to have the property that interrupting/killing/aborting/striking-it-with-lightning at any point during execution leaves you in the least-surprising state we can leave you in, and "arc uninstalled LFS as a side effect" seems highly-surprising, so I'm hesitant to go down that path. Let me take a quick look and see if I can find some other way to trick LFS here -- maybe there's a clever --config flag or something like that.
Thanks Evan! I tried what you suggested but I think GIT_LFS_SKIP_SMUDGE=1 is only for pulling not pushing, since the first push in your suggestion is still uploading to LFS and their code doesn't seem to refer to that env var https://github.com/git-lfs/git-lfs/blob/5f969e6a3e48ecee48b6235adccc42a644e82101/commands/command_push.go, I can't really see any way to disable lfs other than git lfs uninstall; git push; git lfs install. Is that nuts?
Mar 19 2020
That is, the sequence would look something like this, if the description above was muddy:
Part of this change -- pushing to a branch rather than a tag -- breaks a million things and can't come upstream without significant changes elsewhere.
- only run fetch if git for-each-ref is empty
- fix spelling
- handle the case where you need to remove it before adding it
try again with the remote removed from the failure before
try again now that I put in my ssh key
Mar 18 2020
It's true that it has been 12 years, but I'm sure fwrite() will start returning an error code when it encounters a permanent, fatal EPIPE error condition soon.
Mar 11 2020
The issue doesn't appear in the current code base with Firefox 73. That's not unexpected, as the UI has been tweaked several times between 2015 and now.
Mar 9 2020
We also have two separate pieces of ngram extraction code:
For now, I'm going to change the ngram slicing to be character-oriented. This should never be worse than the current behavior, and moves us closer to effective normalization.
This appears to be the unicode normalization chart:
- Fix pattern
Mar 6 2020
If you're affected by this and arc upgrade doesn't work, the easiest fix is to run git pull in arcanist/.
Mar 2 2020
Does using --max-pack-size to reduce the maximum packfile size really let Git "checkpoint" after each packfile, so the process is effectively resumable?
PHI1655 identifies a specific case where enormous packfiles may create problems:
Feb 28 2020
I have some code which runs and looks plausible (i.e., not covered in piles of callback garbage), at least: