Closed Bug 1325490 Opened 3 years ago Closed 2 years ago

Eclipse project generation should give a helpful error message if 'eclipse' is not in the PATH

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox60 fixed)

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: botond, Assigned: botond)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

STR:
  1. Check out a fresh clone of m-c (or mozilla-unified)
  2. Run './mach build'
  3. Run './mach build-backend -b CppEclipse'

This results in the following exception:

$ ./mach build-backend -b CppEclipse
 0:00.08 /home/botond/dev/projects/mozilla/central/obj-x86_64-pc-linux-gnu/_virtualenv/bin/python /home/botond/dev/projects/mozilla/central/obj-x86_64-pc-linux-gnu/config.status --backend CppEclipse
Reticulating splines...
Traceback (most recent call last):
  File "/home/botond/dev/projects/mozilla/central/obj-x86_64-pc-linux-gnu/config.status", line 1230, in <module>
    config_status(**args)
  File "/home/botond/dev/projects/mozilla/central/python/mozbuild/mozbuild/config_status.py", line 150, in config_status
    the_backend.consume(definitions)
  File "/home/botond/dev/projects/mozilla/central/python/mozbuild/mozbuild/backend/base.py", line 143, in consume
    self.consume_finished()
  File "/home/botond/dev/projects/mozilla/central/python/mozbuild/mozbuild/backend/cpp_eclipse.py", line 127, in consume_finished
    self._import_project()
  File "/home/botond/dev/projects/mozilla/central/python/mozbuild/mozbuild/backend/cpp_eclipse.py", line 143, in _import_project
    "-data", self._workspace_dir, "-importAll", self._project_dir])
  File "/usr/lib/python2.7/subprocess.py", line 535, in check_call
    retcode = call(*popenargs, **kwargs)
  File "/usr/lib/python2.7/subprocess.py", line 522, in call
    return Popen(*popenargs, **kwargs).wait()
  File "/usr/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/usr/lib/python2.7/subprocess.py", line 1335, in _execute_child
    raise child_exception
OSError: [Errno 2] No such file or directory
The problems appears to be that eclipse is not in my PATH.

So, not broken, but could give a much better error message.
Summary: Eclipse project generation is broken → Eclipse project generation should give a helpful error message if 'eclipse' is not in the PATH
Eclipse project generation is definitely an under-loved feature. BenWa was the de-facto maintainer. But he's no longer active. I'd love to see someone step up to "own" the feature. As module owner, I'd have no problem delegating review privileges (possibly via sub-module) to people willing to maintain the various IDE integrations. We just need people to step forward...
As a heavy user of Eclipse for C++ development, I've been meaning to spend some time looking at our C++ project generation code. (I still have a post-landing r? flag on bug 973770, which introduced the support, as a reminder to myself to familiarize myself with the code, in preparation for fixing bugs in it / making improvements to it. I just haven't gotten around to it yet...)

I'm happy to park this bug with me for now.
Assignee: nobody → botond
Thank you for taking an interest, Botond! I'm not an Eclipse user, but if there is anything I can do to help, let me know. And I'm sure you know there are plenty of people you can ping in #build.
Comment on attachment 8949152 [details]
Bug 1325490 - Give a helpful error message if eclipse project generation is invoked and 'eclipse' is not in the PATH.

https://reviewboard.mozilla.org/r/218552/#review224348

Sure!
Attachment #8949152 - Flags: review+
Attachment #8949152 - Flags: review?(core-build-config-reviews)
Pushed by bballo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e57bbf2bbcf9
Give a helpful error message if eclipse project generation is invoked and 'eclipse' is not in the PATH. r=nalexander
https://hg.mozilla.org/mozilla-central/rev/e57bbf2bbcf9
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.