[Buildroot] [Bug 11481] New: Docs: Is external.desc required?

[Buildroot] [Bug 11481] New: Docs: Is external.desc required?

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=11481

            Bug ID: 11481
           Summary: Docs: Is external.desc required?
           Product: buildroot
           Version: 2018.08.2
          Hardware: All
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P5
         Component: Other
          Assignee: unassigned at buildroot.uclibc.org
          Reporter: imphil at philipp-wagner.com
                CC: buildroot at uclibc.org
  Target Milestone: ---

I have a minor docs comment.

https://buildroot.org/downloads/manual/manual.html#customize

In the user guide, section "9.1. Recommended directory structure" the directory
tree shows two files for a br2 external tree: 

=====
+-- Config.in (if using a br2-external tree)
+-- external.mk (if using a br2-external tree)
=====

The documentation further down suggests that three files are needed:

=====
9.2.1. Layout of a br2-external tree

A br2-external tree must contain at least those three files, described in the
following chapters:

    external.desc
    external.mk
    Config.in 
=====


Most likely the external.desc file is missing from the top section?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Buildroot] [Bug 11481] Docs: Is external.desc required?

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=11481

Yann E. MORIN <yann.morin.1998@free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |yann.morin.1998 at free.fr
           Assignee|unassigned at buildroot.uclibc |yann.morin.1998 at free.fr
                   |.org                        |
             Status|NEW                         |ASSIGNED

--- Comment #1 from Yann E. MORIN <yann.morin.1998@free.fr> ---
Philipp, All,

Indeed, external.desc is missiing from section 9.1.

Can you send a patch to fix that, to the list, please?

Regards,
Yann E. MORIN.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Buildroot] [Bug 11481] Docs: Is external.desc required?

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=11481

Thomas Petazzoni <thomas.petazzoni@free-electrons.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Thomas Petazzoni <thomas.petazzoni@free-electrons.com> ---
Fixed by
https://git.buildroot.org/buildroot/commit/?id=a6479d6058b75533dd68081d90cec0dbfe652ca4.
Thanks!

-- 
You are receiving this mail because:
You are on the CC list for the bug.