Skip to content

Support perforce as an origin and destination#347

Open
dijit wants to merge 2 commits into
google:masterfrom
dijit:master
Open

Support perforce as an origin and destination#347
dijit wants to merge 2 commits into
google:masterfrom
dijit:master

Conversation

@dijit

@dijit dijit commented Jun 30, 2026

Copy link
Copy Markdown

There's a lot of warts with this currently, this is the patch we're using at Red Notice to sync our git/gerrit repos with our game depots in perforce.

Therefore it's built for specifically our needs;

Still missing to be production ready:

[x] P4TOKEN support
[ ] Dry-Run mode (we don't use it, so we didn't wire it)
[ ] Symlinks and special characters
[ ] SSL Support (incl. hash pinning)
[ ] UTF-8 Support (incl. translation)

If you want me to keep working on this patch I will, it'd be better to have the work upstreamed for me instead of constantly rebasing.

I had to make some minor edits to MODULE.bazel to include p4java (instead of shelling out to a p4 binary)
I also changed the copyrights from our Red Notice ones to be Google ones.

Let me know if I should proceed with the finishing touches or not. We're happy to continue with the patch as-is of course.

@dijit

dijit commented Jun 30, 2026

Copy link
Copy Markdown
Author

I'm going to guess that import/copybara fails because of the new include

MODULE.bazel -> com.perforce:p4java:2026.1.2989454
third_party/BUILD -> //third_party:p4java

Sorry, it's not an error I can clear.

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

Successfully merging this pull request may close these issues.

1 participant