Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fatal: remote error: Git repository not found #16

Open
ryandesign opened this issue May 19, 2021 · 1 comment
Open

fatal: remote error: Git repository not found #16

ryandesign opened this issue May 19, 2021 · 1 comment

Comments

@ryandesign
Copy link

When fetching skia's dependencies prior to building skia, the skia common repo fails to sync because the repository appears to have been deleted from the server:

$ python3.9 tools/git-sync-deps
fatal: remote error: Git repository not found
Exception in thread Thread-2:
Traceback (most recent call last):
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/threading.py", line 954, in _bootstrap_inner
    self.run()
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/threading.py", line 892, in run
    self._target(*self._args, **self._kwargs)
  File "/path/to/skia/tools/git-sync-deps", line 131, in git_checkout_to_directory
    subprocess.check_call(
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/subprocess.py", line 373, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['git', 'clone', '--quiet', '--no-checkout', 'https://skia.googlesource.com/common.git', '/path/to/skia/common']' returned non-zero exit status 128.
Skipping "../src".

This does not prevent the build from completing successfully.

@jusw85
Copy link

jusw85 commented May 20, 2021

It was removed from the skia project on 4th May

google@64e74e6#diff-4cc955bcf0b472353786cc8d8b7d346fb432c091bed099de884885e938392814

dacap pushed a commit that referenced this issue Nov 4, 2021
In the stack trace in the bug, #16 findOrCreateStrike() grabs the
lock, and then #8 Dump() makes calls that grab the lock. This is
only a problem with debug code.

Bug: chromium:1255085

Change-Id: Idcaad8bf2e8c2823bec899ee2b96a74ae666c164
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/455501
Commit-Queue: Herb Derby <[email protected]>
Reviewed-by: Ben Wagner <[email protected]>
dacap pushed a commit that referenced this issue May 15, 2023
I unexpectedly discovered that we can reduce our splat-copy ops
by one instruction by swapping these struct fields. This is
apparently because we can wedge a right-shift by 32 into an add
instruction. (`uxtw` means zero-extend.)

Before (splat_2_constants):
    bcf4: 28 04 40 f9  	ldr	x8, [x1, #8]
    bcf8: 09 fd 60 d3  	lsr	x9, x8, #32          <--- eliminated
    bcfc: 30 01 27 1e  	fmov	s16, w9
    bd00: 10 06 04 4e  	dup.4s	v16, v16[0]
    bd04: 88 40 28 8b  	add	x8, x4, w8, uxtw     <--- changed
    bd08: 10 41 00 ad  	stp	q16, q16, [x8]
    bd0c: 25 0c 41 f8  	ldr	x5, [x1, #16]!
    bd10: a0 00 1f d6  	br	x5

After:
    baa0: 28 04 40 f9  	ldr	x8, [x1, #8]
    baa4: 10 01 27 1e  	fmov	s16, w8
    baa8: 10 06 04 4e  	dup.4s	v16, v16[0]
    baac: 88 80 48 8b  	add	x8, x4, x8, lsr #32
    bab0: 10 41 00 ad  	stp	q16, q16, [x8]
    bab4: 25 0c 41 f8  	ldr	x5, [x1, #16]!
    bab8: a0 00 1f d6  	br	x5

(This also saves an op on Haswell!)

Change-Id: Icea7196b42bc4057d697bbf049d368193d46f27e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/679719
Commit-Queue: John Stiles <[email protected]>
Auto-Submit: John Stiles <[email protected]>
Reviewed-by: Brian Osman <[email protected]>
Commit-Queue: Brian Osman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants