Skip to content
This repository was archived by the owner on Jul 28, 2021. It is now read-only.

Conversation

@jchip
Copy link
Contributor

@jchip jchip commented Dec 1, 2018

  • first fs test that uses pkglock
  • need to add a process.tink._isSelf_ to by pass the self check

@jchip jchip force-pushed the pkglock-lstatsync-test branch from a0aad9f to a733499 Compare December 1, 2018 23:52
@jchip jchip force-pushed the pkglock-lstatsync-test branch from a733499 to f7d5260 Compare December 2, 2018 00:25
@jchip
Copy link
Contributor Author

jchip commented Dec 2, 2018

test was still passing with this commit:

commit bc20c7c68509be6b5d9f6c467fb61d7883f8c4b7
Author: Kat Marchán <kzm@zkat.tech>
Date:   Wed Nov 28 20:26:27 2018 -0800

    refactor(node): move the extensions bit to its own module

2 new commits after that cause the test to fail with:

notice fs fetching require-from-path@1.0.2
ERR! prepare Error: No dependencies found matching filter
ERR! prepare     at Installer.prepare (/Users/xchen11/dev/pub/tink/lib/installer.js:6:1776)
ERR! prepare  { Error: No dependencies found matching filter
ERR! prepare     at Installer.prepare (/Users/xchen11/dev/pub/tink/lib/installer.js:6:1776)
ERR! prepare   stack:
ERR! prepare    'Error: No dependencies found matching filter\n    at Installer.prepare (/Users/xchen11/dev/pub/tink/lib/installer.js:6:1776)' }
test/node/fs-pkglock.js ............................... 0/2 1s
  lstatSync
  not ok fs-pkglock.lstatSync caught Error: No cache entry for tinked-package:2.0.0:require-from-path:1.0.2:https://registry.npmjs.org/require-from-path/-/require-from-path-1.0.2.tgz:sha512-hZrak6935a0RVpx6SFJg59IhRuwWkOBX4VPevL8iyxwoT8ep+ILMziLd1f6s5UCeZ6ckEk06xvfAe2I+3IpUaQ== found in /Users/xchen11/dev/pub/tink/test/.cache

@jchip
Copy link
Contributor Author

jchip commented Jan 8, 2019

@zkat any pointer in the error? I should get a few hours this/next week and will try to resolve this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant