checking out javaparse sources from google code no longer works

RESOLVED FIXED in Firefox 51

Status

()

RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: froydnj, Assigned: hsivonen)

Tracking

Trunk
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

(Whiteboard: btpp-followup-2016-06-06)

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
froydnj@thor:~/src/gecko-dev.git/parser/html/java$ make sync_javaparser
if [ -d javaparser/.svn ] ; \
  then cd javaparser ; svn up ; cd .. ; \
  else \
    rm -rf javaparser ; \
    svn co http://javaparser.googlecode.com/svn/trunk/JavaParser javaparser ; \
  fi
svn: E160013: Unable to connect to a repository at URL 'http://javaparser.googlecode.com/svn/trunk/JavaParser'
svn: E160013: '/svn/trunk/JavaParser' path not found
Makefile:19: recipe for target 'sync_javaparser' failed
make: *** [sync_javaparser] Error 1

This makes modifying the HTML parser difficult for those of us who don't know the magic incantations.
Henri, what should we do here?
Flags: needinfo?(hsivonen)
Whiteboard: btpp-followup-2016-06-06
Comment hidden (mozreview-request)
(Assignee)

Comment 3

2 years ago
(In reply to Andrew Overholt [:overholt] from comment #1)
> Henri, what should we do here?

Let's check out an old tag from GitHub. Migrating to the current version does not seem worthwhile.
Assignee: nobody → hsivonen
Flags: needinfo?(hsivonen)
(Reporter)

Comment 4

2 years ago
mozreview-review
Comment on attachment 8780072 [details]
Bug 1274400 - Check out javaparser from GitHub instead of Google Code.

https://reviewboard.mozilla.org/r/70888/#review68296

WFM, thank you for fixing this!
Attachment #8780072 - Flags: review?(nfroyd) → review+

Comment 5

2 years ago
Pushed by hsivonen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9f38d42002f8
Check out javaparser from GitHub instead of Google Code. r=froydnj

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9f38d42002f8
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
status-firefox49: affected → ---
You need to log in before you can comment on or make changes to this bug.