Skip to content
This repository was archived by the owner on Oct 15, 2024. It is now read-only.

Conversation

@fmeum
Copy link
Member

@fmeum fmeum commented Aug 14, 2020

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates

📜 Description

Let's move the investigation started in #1019 to this fresh PR.

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I formatted the code with the IDE's reformat action (Ctrl + Shift + L/Cmd + Shift + L)
  • I reviewed submitted code
  • I added a CHANGELOG entry if applicable

🔮 Next steps

📸 Screenshots / GIFs

FabianHenneke and others added 2 commits August 14, 2020 08:07
* Add symlink support to JGit

* Fix a typo

Co-authored-by: Harsh Shandilya <me@msfjarvis.dev>
@fmeum
Copy link
Member Author

fmeum commented Aug 14, 2020

I tried to break things by adding and deleting symlinks form the pass-test repo, to no avail. All I found was a simple crash when trying to follow a broken symlink, but no crashes that originate in JGit.

@msfjarvis
Copy link
Member

Now I can't repro it either with the same repo 😕

Here be dragons, I guess.

@msfjarvis msfjarvis added bug and removed in progress labels Aug 14, 2020
@msfjarvis msfjarvis marked this pull request as ready for review August 14, 2020 07:57
@msfjarvis msfjarvis merged commit 087ab54 into develop Aug 14, 2020
@msfjarvis msfjarvis deleted the bug/symlinks_reland branch August 14, 2020 07:57
msfjarvis added a commit to Nosweh/Android-Password-Store that referenced this pull request Aug 20, 2020
* develop:
  build: bump version
  Update CHANGELOG
  Revert "Reland symlink support (android-password-store#1020)"

Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
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.

4 participants