1088520 - "mach ide eclipse" generate empty project on Linux

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

After using "mach ide eclipse" to generate eclipse project on Linux machine, I cannot see any source files in the "Project explorer" in eclipse. However, it works well on Mac OS. I use Eclipse CDT 8.5.

The generated eclipse folder contains:
$ tree .
.
├── gecko
│   └── RunConfigurations
│       └── gecko.launch
├── Gecko
└── RemoteSystemsTempFiles

And the project name I saw in "Project explorer" is "Gecko".

Comment 1

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

I add a print in [1], and got the following log. That "Gecko" folder with capital "G" should be generated by eclipse.

0:28.87 /home/tlin/Projects/gecko-dev/obj-firefox/_virtualenv/bin/python /home/tlin/Projects/gecko-dev/obj-firefox/config.status --backend=CppEclipse
Reticulating splines...
dir_name /home/tlin/Projects/eclipse_obj-firefox/gecko
dir_name /home/tlin/Projects/eclipse_obj-firefox/gecko/.settings
dir_name /home/tlin/Projects/eclipse_obj-firefox/gecko/RunConfigurations
dir_name /home/tlin/Projects/eclipse_obj-firefox/.metadata/.plugins/org.eclipse.core.runtime/.settings
dir_name /home/tlin/Projects/eclipse_obj-firefox/.metadata/.plugins/org.eclipse.cdt.core
Create.
Opening 'Gecko'.
Refreshing '/Gecko'.

[1] http://hg.mozilla.org/mozilla-central/annotate/9781037ac408/python/mozbuild/mozbuild/backend/cpp_eclipse.py#l77

Comment 2

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

Attachment: Fix empty eclipse project generated on Linux.

Use "gecko" with small letter 'g' for both project dir and the project
name in .project.

Comment 3

[Mike Hommey [:glandium]]

Ah the joys of case-insensitive file systems.

Comment 4

[Mike Hommey [:glandium]]

Comment on attachment 8510871 [details] [diff] [review]
Fix empty eclipse project generated on Linux.

Review of attachment 8510871 [details] [diff] [review]:
-----------------------------------------------------------------

::: python/mozbuild/mozbuild/backend/cpp_eclipse.py
@@ +26,5 @@
>      def _init(self):
>          CommonBackend._init(self)
>  
>          self._paths_to_defines = {}
> +        self._project_name = 'gecko'

Can you make it 'Gecko'?

Comment 5

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

Attachment: Fix empty eclipse project generated on Linux. r=glandium (carry r+)

Addressed comment 4.

Use "Gecko" with captical letter 'G' for both project dir and the
project name in .project.

Comment 6

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

This change is NPOTB. Thanks.

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8a799dd8a11c

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/8a799dd8a11c