Misc fixes/improvements

PATCH Applied Otavio Salvador 6 participants 21 messages Dec 8, 2011 – Feb 7, 2012

lore

Share & tools

lore

Otavio Salvador

author

2011-12-08 15:47 UTC #0 lore raw

cover Misc fixes/improvements

The following changes since commit 60f0b866b4b7c4aa337a2ff764455741a73665da:

package.bbclass: Adding license collection (2011-12-08 15:23:58 +0000)

are available in the git repository at:
git://github.com/OSSystems/oe-core master
https://github.com/OSSystems/oe-core/tree/HEAD

Otavio Salvador (3):
cross.bbclass: add virtclass handler
dosfstools: update native to 2.11 scripts/send-pull-request: allow sending of patches without a cover
letter

 meta/classes/cross.bbclass                         |   50 +++++++++++++
 .../dosfstools/dosfstools-native_2.10.bb           |   19 -----
 .../dosfstools/dosfstools/2.6.20-syscall.patch     |   72 -------------------
 .../dosfstools/dosfstools-2.10-kernel-2.6.patch    |   74 --------------------
 .../dosfstools/dosfstools/mkdosfs-bootcode.patch   |   68 ++++++++----------
 .../dosfstools/dosfstools/mkdosfs-dir.patch        |   62 ++++++++---------
 .../recipes-devtools/dosfstools/dosfstools_2.10.bb |   24 ------
 .../recipes-devtools/dosfstools/dosfstools_2.11.bb |   11 ++-
 scripts/send-pull-request                          |   21 +++---
 9 files changed, 131 insertions(+), 270 deletions(-)
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools_2.10.bb

Signature · 1 line

1.7.2.5

Otavio Salvador

author

2011-12-08 15:47 UTC #1 lore raw

PATCH 1/3 cross.bbclass: add virtclass handler

Allow use of BBCLASSEXTEND with 'cross' and use of virtclass-cross in recipes.

Signed-off-by Otavio Salvador <[email protected]>

---
 meta/classes/cross.bbclass |   50 ++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 50 insertions(+), 0 deletions(-)

meta/classes/cross.bbclass +50 -0

index 5681ab9..a3b8e10 100644
@@ -55,6 +55,56 @@ libexecdir = "${exec_prefix}/libexec/${CROSS_TARGET_SYS_DIR}"
 
 do_populate_sysroot[sstate-inputdirs] = "${SYSROOT_DESTDIR}/${STAGING_DIR_NATIVE}"
 
+python cross_virtclass_handler () {
+    if not isinstance(e, bb.event.RecipePreFinalise):
+        return
+
+    classextend = e.data.getVar('BBCLASSEXTEND', True) or ""
+    if "cross" not in classextend:
+        return
+
+    pn = e.data.getVar("PN", True)
+    if not pn.endswith("-cross"):
+        return
+
+    def map_dependencies(varname, d, suffix = ""):
+        if suffix:
+            varname = varname + "_" + suffix
+        deps = d.getVar(varname, True)
+        if not deps:
+            return
+        deps = bb.utils.explode_deps(deps)
+        newdeps = []
+        for dep in deps:
+            if dep.endswith("-cross"):
+                newdeps.append(dep.replace("-cross", "-cross"))
+            elif not dep.endswith("-cross"):
+                newdeps.append(dep + "-cross")
+            else:
+                newdeps.append(dep)
+        bb.data.setVar(varname, " ".join(newdeps), d)
+
+    map_dependencies("DEPENDS", e.data)
+    for pkg in (e.data.getVar("PACKAGES", True).split() + [""]):
+        map_dependencies("RDEPENDS", e.data, pkg)
+        map_dependencies("RRECOMMENDS", e.data, pkg)
+        map_dependencies("RSUGGESTS", e.data, pkg)
+        map_dependencies("RPROVIDES", e.data, pkg)
+        map_dependencies("RREPLACES", e.data, pkg)
+
+    provides = e.data.getVar("PROVIDES", True)
+    for prov in provides.split():
+        if prov.find(pn) != -1:
+            continue
+        if not prov.endswith("-cross"):
+            provides = provides.replace(prov, prov + "-cross")
+    e.data.setVar("PROVIDES", provides)
+
+    bb.data.setVar("OVERRIDES", e.data.getVar("OVERRIDES", False) + ":virtclass-cross", e.data)
+}
+
+addhandler cross_virtclass_handler
+
 do_install () {
 	oe_runmake 'DESTDIR=${D}' install
 }

Signature · 1 line

1.7.2.5

Richard Purdie

2011-12-08 22:16 UTC #2 lore raw

Re: [PATCH 1/3] cross.bbclass: add virtclass handler

Quoted 29 lines · Otavio Salvador

Allow use of BBCLASSEXTEND with 'cross' and use of virtclass-cross in recipes.

Signed-off-by Otavio Salvador <[email protected]>

---
 meta/classes/cross.bbclass |   50 ++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 50 insertions(+), 0 deletions(-)

meta/classes/cross.bbclass +13 -0

index 5681ab9..a3b8e10 100644
@@ -55,6 +55,56 @@ libexecdir = "${exec_prefix}/libexec/${CROSS_TARGET_SYS_DIR}"
 
 do_populate_sysroot[sstate-inputdirs] = "${SYSROOT_DESTDIR}/${STAGING_DIR_NATIVE}"
 
+python cross_virtclass_handler () {
+    if not isinstance(e, bb.event.RecipePreFinalise):
+        return
+
+    classextend = e.data.getVar('BBCLASSEXTEND', True) or ""
+    if "cross" not in classextend:
+        return
+
+    pn = e.data.getVar("PN", True)
+    if not pn.endswith("-cross"):
+        return
+
+    def map_dependencies(varname, d, suffix = ""):

meta/classes/cross.bbclass libexecdir = "${exec_prefix}/libexec/${CROSS_TARGET_SYS_DIR}" · line 80

Quoted 10 lines

meta/classes/cross.bbclass +10 -0

+        if suffix:
+            varname = varname + "_" + suffix
+        deps = d.getVar(varname, True)
+        if not deps:
+            return
+        deps = bb.utils.explode_deps(deps)
+        newdeps = []
+        for dep in deps:
+            if dep.endswith("-cross"):
+                newdeps.append(dep.replace("-cross", "-cross"))

I know you did a copy, search, replace and paste from native.bbclass but you could at least sanity check the result :)

meta/classes/cross.bbclass libexecdir = "${exec_prefix}/libexec/${CROSS_TARGET_SYS_DIR}" · line 85

Quoted 5 lines

meta/classes/cross.bbclass +5 -0

+            elif not dep.endswith("-cross"):
+                newdeps.append(dep + "-cross")
+            else:
+                newdeps.append(dep)
+        bb.data.setVar(varname, " ".join(newdeps), d)

The logic here is just totally wrong. You can't ever hit the else case and the paths you do hit are nonsensical.

Quoted 6 lines

meta/classes/cross.bbclass +6 -0

+
+    map_dependencies("DEPENDS", e.data)
+    for pkg in (e.data.getVar("PACKAGES", True).split() + [""]):
+        map_dependencies("RDEPENDS", e.data, pkg)
+        map_dependencies("RRECOMMENDS", e.data, pkg)
+        map_dependencies("RSUGGESTS", e.data, pkg)

meta/classes/cross.bbclass libexecdir = "${exec_prefix}/libexec/${CROSS_TARGET_SYS_DIR}" · line 101

Quoted 10 lines

meta/classes/cross.bbclass +10 -0

+        map_dependencies("RPROVIDES", e.data, pkg)
+        map_dependencies("RREPLACES", e.data, pkg)
+
+    provides = e.data.getVar("PROVIDES", True)
+    for prov in provides.split():
+        if prov.find(pn) != -1:
+            continue
+        if not prov.endswith("-cross"):
+            provides = provides.replace(prov, prov + "-cross")
+    e.data.setVar("PROVIDES", provides)

My suggestion is that you should just remove the DEPENDS and PROVIDES mapping code above and do it manually using the virtclass-cross override in the recipe. The above code can't have been doing much that is sensible anyway.

Cheers,

Richard

Quoted 8 lines

meta/classes/cross.bbclass +5 -0

+    bb.data.setVar("OVERRIDES", e.data.getVar("OVERRIDES", False) + ":virtclass-cross", e.data)
+}
+
+addhandler cross_virtclass_handler
+
 do_install () {
 	oe_runmake 'DESTDIR=${D}' install
 }

Otavio Salvador

author

2012-02-07 19:36 UTC #3 lore raw

Re: [PATCH 1/3] cross.bbclass: add virtclass handler

Quoted 8 lines · Richard Purdie

meta/classes/cross.bbclass libexecdir = "${exec_prefix}/libexec/${CROSS_TARGET_SYS_DIR}" · line 85
Quoted 5 lines
meta/classes/cross.bbclass +5 -0
+            elif not dep.endswith("-cross"):
+                newdeps.append(dep + "-cross")
+            else:
+                newdeps.append(dep)
+        bb.data.setVar(varname, " ".join(newdeps), d)
The logic here is just totally wrong. You can't ever hit the else case and the paths you do hit are nonsensical.

Fixed it.

Quoted 16 lines

+ map_dependencies("DEPENDS", e.data)

Quoted 4 lines

meta/classes/cross.bbclass +4 -0

+    for pkg in (e.data.getVar("PACKAGES", True).split() + [""]):
+        map_dependencies("RDEPENDS", e.data, pkg)
+        map_dependencies("RRECOMMENDS", e.data, pkg)
+        map_dependencies("RSUGGESTS", e.data, pkg)

meta/classes/cross.bbclass libexecdir = "${exec_prefix}/libexec/${CROSS_TARGET_SYS_DIR}" · line 101

Quoted 10 lines

meta/classes/cross.bbclass +10 -0

+        map_dependencies("RPROVIDES", e.data, pkg)
+        map_dependencies("RREPLACES", e.data, pkg)
+
+    provides = e.data.getVar("PROVIDES", True)
+    for prov in provides.split():
+        if prov.find(pn) != -1:
+            continue
+        if not prov.endswith("-cross"):
+            provides = provides.replace(prov, prov + "-cross")
+    e.data.setVar("PROVIDES", provides)

Quoted 5 lines

My suggestion is that you should just remove the DEPENDS and PROVIDES mapping code above and do it manually using the virtclass-cross override in the recipe. The above code can't have been doing much that is sensible anyway.

Right. I did it and will send an updated patch tomorrow after a from scratch build.

Signature · 3 lines

Otavio Salvador                             O.S. Systems
E-mail: [email protected]  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Otavio Salvador

author

2011-12-08 15:48 UTC #4 lore raw

PATCH 2/3 dosfstools: update native to 2.11

In reply to Otavio Salvador

This unify recipes for target and native builds and also drops the the already merged patches.

Signed-off-by Otavio Salvador <[email protected]>

---
 .../dosfstools/dosfstools-native_2.10.bb           |   19 -----
 .../dosfstools/dosfstools/2.6.20-syscall.patch     |   72 -------------------
 .../dosfstools/dosfstools-2.10-kernel-2.6.patch    |   74 --------------------
 .../dosfstools/dosfstools/mkdosfs-bootcode.patch   |   68 ++++++++----------
 .../dosfstools/dosfstools/mkdosfs-dir.patch        |   62 ++++++++---------
 .../recipes-devtools/dosfstools/dosfstools_2.10.bb |   24 ------
 .../recipes-devtools/dosfstools/dosfstools_2.11.bb |   11 ++-
 7 files changed, 69 insertions(+), 261 deletions(-)
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools_2.10.bb

meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb deleted +0 ~~-19~~

deleted file mode 100644
index 91ff11f..0000000
@@ -1,19 +0,0 @@
-# dosfstools-native OE build file
-# Copyright (C) 2004-2006, Advanced Micro Devices, Inc.  All Rights Reserved
-# Released under the MIT license (see packages/COPYING)
-
-require dosfstools_${PV}.bb
-
-PR="r5"
-
-SRC_URI = "ftp://ftp.uni-erlangen.de/pub/Linux/LOCAL/dosfstools/dosfstools-${PV}.src.tar.gz \
-	file://mkdosfs-bootcode.patch \
-	file://mkdosfs-dir.patch \
-	file://alignment_hack.patch \
-	file://dosfstools-2.10-kernel-2.6.patch \
-	file://msdos_fat12_undefined.patch \
-	file://dosfstools-msdos_fs-types.patch \
-	file://include-linux-types.patch \
-	file://2.6.20-syscall.patch"
-
-inherit native

meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch deleted +0 ~~-72~~

72 lines not on this page · raw message

meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch deleted +0 ~~-74~~

74 lines not on this page · raw message

meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-bootcode.patch +31 ~~-37~~

166 lines not on this page · raw message

meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +30 ~~-32~~

165 lines not on this page · raw message

meta/recipes-devtools/dosfstools/dosfstools_2.10.bb deleted +0 ~~-24~~

deleted file mode 100644
index c23c02d..0000000
@@ -1,24 +0,0 @@
-# dosfstools OE build file
-# Copyright (C) 2004-2006, Advanced Micro Devices, Inc.  All Rights Reserved
-# Released under the MIT license (see packages/COPYING)
-
-DESCRIPTION = "DOS FAT Filesystem Utilities"
-
-SECTION = "base"
-LICENSE = "GPLv2"
-LIC_FILES_CHKSUM = "file://mkdosfs/COPYING;md5=cbe67f08d6883bff587f615f0cc81aa8"
-PR = "r3"
-
-SRC_URI = "ftp://ftp.uni-erlangen.de/pub/Linux/LOCAL/dosfstools/dosfstools-${PV}.src.tar.gz \
-	   file://alignment_hack.patch \
-	   file://dosfstools-2.10-kernel-2.6.patch \
-           file://msdos_fat12_undefined.patch \
-	   file://include-linux-types.patch"
-
-SRC_URI[md5sum] = "59a02f311a891af8787c4c9e28c6b89b"
-SRC_URI[sha256sum] = "55a7b2f5ea4506bde935ee3145573e1773427fc72283a36796c7c2cf861dd064"
-
-do_install () {
-	oe_runmake "PREFIX=${D}" "SBINDIR=${D}${sbindir}" \
-		   "MANDIR=${D}${mandir}/man8" install
-}

meta/recipes-devtools/dosfstools/dosfstools_2.11.bb +8 -3

index 944d873..7d0080c 100644
@@ -7,12 +7,15 @@ DESCRIPTION = "DOS FAT Filesystem Utilities"
 SECTION = "base"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://mkdosfs/COPYING;md5=cbe67f08d6883bff587f615f0cc81aa8"
-PR = "r0"
+PR = "r1"
 
 SRC_URI = "ftp://ftp.uni-erlangen.de/pub/Linux/LOCAL/dosfstools/dosfstools-${PV}.src.tar.gz \
-	   file://alignment_hack.patch \
+           file://mkdosfs-bootcode.patch \
+           file://mkdosfs-dir.patch \
+           file://alignment_hack.patch \
            file://msdos_fat12_undefined.patch \
-	   file://include-linux-types.patch"
+           file://dosfstools-msdos_fs-types.patch \
+           file://include-linux-types.patch"
 
 SRC_URI[md5sum] = "407d405ade410f7597d364ab5dc8c9f6"
 SRC_URI[sha256sum] = "0eac6d12388b3d9ed78684529c1b0d9346fa2abbe406c4d4a3eb5a023c98a484"

@@ -21,3 +24,5 @@ do_install () {
 	oe_runmake "PREFIX=${D}" "SBINDIR=${D}${sbindir}" \
 		   "MANDIR=${D}${mandir}/man8" install
 }
+
+BBCLASSEXTEND = "native"

Signature · 1 line

1.7.2.5

Kamble, Nitin A

2011-12-09 19:15 UTC #5 lore raw

Re: [PATCH 2/3] dosfstools: update native to 2.11

<Ack>
I verified it is building fine for all arches.

Nitin

Quoted 785 lines

-----Original Message-----
From: [email protected] [mailto:[email protected]] On Behalf Of Otavio Salvador
Sent: Thursday, December 08, 2011 7:41 AM
To: [email protected]
Subject: [OE-core] [PATCH 2/3] dosfstools: update native to 2.11
This unify recipes for target and native builds and also drops the the already merged patches.
Signed-off-by Otavio Salvador <[email protected]>
---
 .../dosfstools/dosfstools-native_2.10.bb           |   19 -----
 .../dosfstools/dosfstools/2.6.20-syscall.patch     |   72 ------------
-------
 .../dosfstools/dosfstools-2.10-kernel-2.6.patch    |   74 ------------
--------
 .../dosfstools/dosfstools/mkdosfs-bootcode.patch   |   68 ++++++++----
------
 .../dosfstools/dosfstools/mkdosfs-dir.patch        |   62 ++++++++----
-----
 .../recipes-devtools/dosfstools/dosfstools_2.10.bb |   24 ------
 .../recipes-devtools/dosfstools/dosfstools_2.11.bb |   11 ++-
 7 files changed, 69 insertions(+), 261 deletions(-)
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools-
native_2.10.bb
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/2.6.20-
syscall.patch
 delete mode 100644 meta/recipes-
devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools_2.10.bb
meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb deleted +0 -2
deleted file mode 100644
index 91ff11f..0000000
@@ -1,19 +0,0 @@
-# dosfstools-native OE build file
-# Copyright (C) 2004-2006, Advanced Micro Devices, Inc.  All Rights
Reserved
meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb deleted +0 -7
-# Released under the MIT license (see packages/COPYING)
-
-require dosfstools_${PV}.bb
-
-PR="r5"
-
-SRC_URI = "ftp://ftp.uni-
erlangen.de/pub/Linux/LOCAL/dosfstools/dosfstools-${PV}.src.tar.gz \
meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb deleted +0 ~~-10~~
-     file://mkdosfs-bootcode.patch \
-     file://mkdosfs-dir.patch \
-     file://alignment_hack.patch \
-     file://dosfstools-2.10-kernel-2.6.patch \
-     file://msdos_fat12_undefined.patch \
-     file://dosfstools-msdos_fs-types.patch \
-     file://include-linux-types.patch \
-     file://2.6.20-syscall.patch"
-
-inherit native
(quoted diff) +0 -0
syscall.patch b/meta/recipes-devtools/dosfstools/dosfstools/2.6.20- syscall.patch
deleted file mode 100644
index 4460f06..0000000
meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch deleted +1 ~~-13~~
@@ -1,72 +0,0 @@
-Only use the system's llseek().
-
-Upstream-Status: Inappropriate [licensing]
-We're tracking an old release of dosfstools due to licensing issues.
-
-Signed-off-by: Scott Garman <[email protected]>
-
-Index: dosfstools-2.10/dosfsck/io.c
-===================================================================
---- dosfstools-2.10.orig/dosfsck/io.c        2007-06-07 16:15:52.000000000
+0200
-+++ dosfstools-2.10/dosfsck/io.c     2007-06-07 16:16:06.000000000 +0200
-@@ -42,28 +42,11 @@
- /* Use the _llseek system call directly, because there (once?) was a
bug in
- * the glibc implementation of it. */
- #include <linux/unistd.h>
--#if defined __alpha || defined __ia64__ || defined __s390x__ || defined __x86_64__ || defined __ppc64__
- /* On alpha, the syscall is simply lseek, because it's a 64 bit system. */
- static loff_t llseek( int fd, loff_t offset, int whence )
- {
- return lseek(fd, offset, whence);
- }
--#else
--# ifndef __NR__llseek
--# error _llseek system call not present
--# endif
--static _syscall5( int, _llseek, uint, fd, ulong, hi, ulong, lo,
-- loff_t *, res, uint, wh );
--
--static loff_t llseek( int fd, loff_t offset, int whence ) --{
-- loff_t actual;
--
-- if (_llseek(fd, offset>>32, offset&0xffffffff, &actual, whence)
!= 0)
-- return (loff_t)-1;
-- return actual;
--}
--#endif
meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch deleted +0 -6
-
-
- void fs_open(char *path,int rw)
-Index: dosfstools-2.10/mkdosfs/mkdosfs.c
-===================================================================
---- dosfstools-2.10.orig/mkdosfs/mkdosfs.c   2007-06-07
16:15:11.000000000 +0200
-+++ dosfstools-2.10/mkdosfs/mkdosfs.c 2007-06-07 16:15:30.000000000
+0200
-@@ -116,27 +116,11 @@
- /* Use the _llseek system call directly, because there (once?) was a bug in
- * the glibc implementation of it. */
- #include <linux/unistd.h>
--#if defined __alpha || defined __ia64__ || defined __s390x__ || defined __x86_64__ || defined __ppc64__
- /* On alpha, the syscall is simply lseek, because it's a 64 bit system. */
- static loff_t llseek( int fd, loff_t offset, int whence )
- {
- return lseek(fd, offset, whence);
- }
--#else
--# ifndef __NR__llseek
--# error _llseek system call not present
--# endif
--static _syscall5( int, _llseek, uint, fd, ulong, hi, ulong, lo,
-- loff_t *, res, uint, wh );
--static loff_t llseek( int fd, loff_t offset, int whence ) --{
-- loff_t actual;
--
-- if (_llseek(fd, offset>>32, offset&0xffffffff, &actual, whence)
!= 0)
-- return (loff_t)-1;
-- return actual;
--}
--#endif
-
- #define ROUND_UP(value, divisor) (value + (divisor - (value % divisor))) / divisor
meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch deleted +0 -1
-
(quoted diff) +0 -0
2.10-kernel-2.6.patch b/meta/recipes- devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch deleted file mode 100644
index 0c9230f..0000000
--- a/meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10- kernel-2.6.patch
+++ /dev/null
meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch deleted +0 -8
@@ -1,74 +0,0 @@
-Submitted By: Jim Gifford (jim at linuxfromscratch dot org)
-Date: 2004-02-09
-Initial Package Version: 2.6
-Origin: Jim Gifford
-Upstream-Status: Accepted
-Description: Fixes Compile Issues with the 2.6 Kernel
-
---- dosfstools-2.10/dosfsck/common.h.orig    2004-02-09
18:37:59.056737458 +0000
-+++ dosfstools-2.10/dosfsck/common.h 2004-02-09 18:38:18.333392952 +0000
-@@ -2,6 +2,13 @@
meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch deleted +0 -9
-
- /* Written 1993 by Werner Almesberger */
-
-+#include <linux/version.h>
-+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
-+       #define __KERNEL__
-+       #include <asm/types.h>
-+       #undef __KERNEL__
-+       #define MSDOS_FAT12 4084 /* maximum number of clusters in a 12
bit FAT */
-+#endif
-
- #ifndef _COMMON_H
- #define _COMMON_H
---- dosfstools-2.10/dosfsck/file.c.orig 2004-02-09 18:40:52.016728845
+0000
-+++ dosfstools-2.10/dosfsck/file.c 2004-02-09 18:40:03.665117865 +0000
-@@ -15,6 +15,14 @@
- #define _LINUX_STAT_H /* hack to avoid inclusion of
<linux/stat.h> */
- #define _LINUX_STRING_H_ /* hack to avoid inclusion of
<linux/string.h>*/
- #define _LINUX_FS_H /* hack to avoid inclusion of
<linux/fs.h> */
-+
-+#include <linux/version.h>
-+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
-+       #define __KERNEL__
-+       #include <asm/types.h>
-+       #undef __KERNEL__
-+#endif
-+
- #include <linux/msdos_fs.h>
-
- #include "common.h"
---- dosfstools-2.10/dosfsck/dosfsck.h.orig 2004-02-09
18:57:11.022870974 +0000
-+++ dosfstools-2.10/dosfsck/dosfsck.h 2004-02-09 18:56:20.628614393
+0000
-@@ -13,6 +13,15 @@
- #define _LINUX_STAT_H /* hack to avoid inclusion of
<linux/stat.h> */
- #define _LINUX_STRING_H_ /* hack to avoid inclusion of
<linux/string.h>*/
- #define _LINUX_FS_H /* hack to avoid inclusion of
<linux/fs.h> */
-+
-+#include <linux/version.h>
-+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
-+       #define __KERNEL__
-+       #include <asm/types.h>
-+       #include <asm/byteorder.h>
-+       #undef __KERNEL__
-+#endif
-+
- #include <linux/msdos_fs.h>
-
- /* 2.1 kernels use le16_to_cpu() type functions for CF_LE_W & Co., but don't
---- dosfstools-2.10/mkdosfs/mkdosfs.c.orig 2004-02-09
18:31:41.997157413 +0000
-+++ dosfstools-2.10/mkdosfs/mkdosfs.c 2004-02-09 18:34:07.311945252
+0000
-@@ -66,6 +66,13 @@
- #include <time.h>
- #include <errno.h>
meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch deleted +0 ~~-11~~
-
-+#include <linux/version.h>
-+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
-+       #define __KERNEL__
-+       #include <asm/types.h>
-+       #undef __KERNEL__
-+#endif
-+
- #if __BYTE_ORDER == __BIG_ENDIAN
-
- #include <asm/byteorder.h>
(quoted diff) +0 -0
bootcode.patch b/meta/recipes-devtools/dosfstools/dosfstools/mkdosfs- bootcode.patch
index dd5cc7f..ae21bee 100644
--- a/meta/recipes-devtools/dosfstools/dosfstools/mkdosfs- bootcode.patch
+++ b/meta/recipes-devtools/dosfstools/dosfstools/mkdosfs- bootcode.patch
(quoted diff) +0 -0
@@ -5,9 +5,10 @@ We're tracking an old release of dosfstools due to
licensing issues.
Signed-off-by Scott Garman <[email protected]>
-diff -urN dosfstools-2.10.orig/mkdosfs/ChangeLog dosfstools- 2.10/mkdosfs/ChangeLog
---- dosfstools-2.10.orig/mkdosfs/ChangeLog 1997-06-18
03:09:38.000000000 -0700
-+++ dosfstools-2.10/mkdosfs/ChangeLog 2004-08-02 20:57:57.734939816
-0700
+Index: dosfstools-2.11/mkdosfs/ChangeLog
+===================================================================
+--- dosfstools-2.11.orig/mkdosfs/ChangeLog   1997-06-18
10:09:38.000000000 +0000
++++ dosfstools-2.11/mkdosfs/ChangeLog 2011-12-06 12:14:23.634011558
+0000
@@ -1,3 +1,14 @@
+19th June 2003 Sam Bingner ([email protected])
+
(quoted diff) +0 -0
@@ -23,10 +24,11 @@ diff -urN dosfstools-2.10.orig/mkdosfs/ChangeLog
dosfstools-2.10/mkdosfs/ChangeL
28th January 1995 H. Peter Anvin ([email protected])
Better algorithm to select cluster sizes on large filesystems.
-diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.8 dosfstools- 2.10/mkdosfs/mkdosfs.8
---- dosfstools-2.10.orig/mkdosfs/mkdosfs.8 2003-05-15
11:28:28.000000000 -0700
-+++ dosfstools-2.10/mkdosfs/mkdosfs.8 2004-08-02 20:57:57.735939664
-0700
-@@ -40,6 +40,10 @@
+Index: dosfstools-2.11/mkdosfs/mkdosfs.8
+===================================================================
+--- dosfstools-2.11.orig/mkdosfs/mkdosfs.8   2004-02-25
19:36:07.000000000 +0000
++++ dosfstools-2.11/mkdosfs/mkdosfs.8 2011-12-06 12:19:54.777888434
+0000
+@@ -44,6 +44,10 @@
.I message-file
]
[
(quoted diff) +0 -0
@@ -37,7 +39,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.8
dosfstools-2.10/mkdosfs/mkdosfs
.B \-n
.I volume-name
]
-@@ -155,6 +159,18 @@
+@@ -165,6 +169,18 @@
carriage return-line feed combinations, and tabs have been expanded. If the filename is a hyphen (-), the text is taken from standard
input.
.TP
(quoted diff) +0 -0
@@ -56,21 +58,22 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.8
dosfstools-2.10/mkdosfs/mkdosfs
.BI \-n " volume-name"
Sets the volume name (label) of the filesystem. The volume name can be up to 11 characters long. The default is no label.
-@@ -188,8 +204,9 @@
+@@ -198,8 +214,9 @@
simply will not support it ;)
.SH AUTHOR
Dave Hudson - <[email protected]>; modified by Peter Anvin
-<[email protected]>. Fixes and additions by Roman Hodek
--<[email protected]> for Debian/GNU Linux. +-<[email protected]> for Debian/GNU Linux.
+<[email protected]> and Sam Bingner <[email protected]>. Fixes and +additions by Roman Hodek <[email protected]> +for Debian/GNU Linux.
.SH ACKNOWLEDGEMENTS
.B mkdosfs
is based on code from
-diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c dosfstools- 2.10/mkdosfs/mkdosfs.c
---- dosfstools-2.10.orig/mkdosfs/mkdosfs.c 2003-06-14
13:07:08.000000000 -0700
-+++ dosfstools-2.10/mkdosfs/mkdosfs.c 2004-08-02 20:57:57.736939512
-0700
+Index: dosfstools-2.11/mkdosfs/mkdosfs.c
+===================================================================
+--- dosfstools-2.11.orig/mkdosfs/mkdosfs.c   2005-03-12
16:12:16.000000000 +0000
++++ dosfstools-2.11/mkdosfs/mkdosfs.c 2011-12-06 12:27:55.121886076
+0000
@@ -24,6 +24,12 @@
- New options -A, -S, -C
- Support for filesystems > 2GB
(quoted diff) +0 -0
@@ -84,7 +87,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
Copying: Copyright 1993, 1994 David Hudson
([email protected])
-@@ -167,6 +173,8 @@
+@@ -153,6 +159,8 @@
#define FAT_BAD 0x0ffffff7
  #define MSDOS_EXT_SIGN 0x29 /* extended boot sector signature */
(quoted diff) +0 -0
@@ -93,7 +96,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
#define MSDOS_FAT12_SIGN "FAT12 " /* FAT12 filesystem signature
*/
#define MSDOS_FAT16_SIGN "FAT16 " /* FAT16 filesystem signature
*/
#define MSDOS_FAT32_SIGN "FAT32 " /* FAT32 filesystem signature
*/
-@@ -188,6 +196,8 @@
+@@ -175,6 +183,8 @@
#define BOOTCODE_SIZE 448
#define BOOTCODE_FAT32_SIZE 420
(quoted diff) +0 -0
@@ -102,7 +105,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
/* __attribute__ ((packed)) is used on all structures to make gcc
ignore any
* alignments */
-@@ -215,7 +225,7 @@
+@@ -202,7 +212,7 @@
    __u16         fat_length; /* sectors/FAT */
    __u16         secs_track; /* sectors per track */
    __u16         heads;              /* number of heads */
(quoted diff) +0 -0
@@ -111,7 +114,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
    __u32         total_sect; /* number of sectors (if sectors == 0) */
    union {
      struct {
-@@ -298,6 +308,8 @@
+@@ -285,6 +295,8 @@
/* Global variables - the root of all evil :-) - see these and weep!
*/
(quoted diff) +0 -0
@@ -120,7 +123,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
static char *program_name = "mkdosfs"; /* Name of the program */
static char *device_name = NULL; /* Name of the device on which to
create the filesystem */
static int atari_format = 0; /* Use Atari variation of MS-DOS FS
format */
-@@ -842,6 +854,12 @@
+@@ -837,6 +849,12 @@
      vi->volume_id[2] = (unsigned char) ((volume_id & 0x00ff0000) >>
16);
      vi->volume_id[3] = (unsigned char) (volume_id >> 24);
    }
(quoted diff) +0 -0
@@ -133,16 +136,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
    if (!atari_format) {
      memcpy(vi->volume_label, volume_name, 11);
-@@ -886,7 +904,7 @@
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-bootcode.patch +1 -9
-     printf( "Using %d reserved sectors\n", reserved_sectors );
-   bs.fats = (char) nr_fats;
-   if (!atari_format || size_fat == 32)
--    bs.hidden = CT_LE_L(0);
-+    bs.hidden = bs.secs_track;
-   else
-     /* In Atari format, hidden is a 16 bit field */
-     memset( &bs.hidden, 0, 2 );
-@@ -1358,6 +1376,32 @@
+@@ -1362,6 +1380,32 @@
     * dir area on FAT12/16, and the first cluster on FAT32. */
    writebuf( (char *) root_dir, size_root_dir, "root directory" );
@@ -156,9 +150,9 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
+ seekto( 512*2, "third sector" );
+ if (backup_boot != 0) {
+ writebuf( template_boot_code+512*2, backup_boot*sector_size -
512*2, "data to backup boot" );
-+ seekto( backup_boot*sector_size, "backup boot sector" );
++ seekto( backup_boot*sector_size, "backup boot sector" );
+ writebuf( template_boot_code, 3, "backup jmpBoot" );
-+ seekto( backup_boot*sector_size+0x5a, "backup boot sector boot
area" );
++ seekto( backup_boot*sector_size+0x5a, "backup boot sector
boot area" );
+ writebuf( template_boot_code+0x5a, 420, "backup boot sector
boot area" );
+ seekto( (backup_boot+2)*sector_size, "sector following backup
code" );
+ writebuf( template_boot_code+(backup_boot+2)*sector_size,
(reserved_sectors-backup_boot-2)*512, "remaining data" );
(quoted diff) +0 -0
@@ -172,28 +166,28 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
+ }
+ }
+
+ if (blank_sector) free( blank_sector );
if (info_sector) free( info_sector );
free (root_dir); /* Free up the root directory space from
setup_tables */
- free (fat); /* Free up the fat table space reserved during
setup_tables */
-@@ -1371,7 +1415,7 @@
+@@ -1376,7 +1420,7 @@
  {
    fatal_error("\
Usage: mkdosfs [-A] [-c] [-C] [-v] [-I] [-l bad-block-file] [-b
backup-boot-sector]\n\
- [-m boot-msg-file] [-n volume-name] [-i volume-id]\n\
+ [-m boot-msg-file] [-n volume-name] [-i volume-id] [-B
bootcode]\n\
[-s sectors-per-cluster] [-S logical-sector-size] [-f number-
of-FATs]\n\
(quoted diff) +1 -1
-        [-F fat-size] [-r root-dir-entries] [-R reserved-sectors]\n\
+        [-h hidden-sectors] [-F fat-size] [-r root-dir-entries] [-R
reserved-sectors]\n\
         /dev/name [blocks]\n");
-@@ -1433,7 +1477,7 @@
+@@ -1439,7 +1483,7 @@
    printf ("%s " VERSION " (" VERSION_DATE ")\n",
         program_name);
-- while ((c = getopt (argc, argv, "AcCf:F:Ii:l:m:n:r:R:s:S:v")) != EOF)
-+ while ((c = getopt (argc, argv, "AcCf:F:Ii:l:m:n:r:R:s:S:v:B:b")) != EOF)
+- while ((c = getopt (argc, argv, "AbcCf:F:Ii:l:m:n:r:R:s:S:h:v")) != EOF)
++ while ((c = getopt (argc, argv, "AbcCf:F:Ii:l:m:n:r:R:s:S:v:B:")) != EOF)
      /* Scan the command line for options */
      switch (c)
        {
-@@ -1494,6 +1538,51 @@
+@@ -1509,6 +1553,51 @@
      listfile = optarg;
      break;
(quoted diff) +0 -0
dir.patch b/meta/recipes-devtools/dosfstools/dosfstools/mkdosfs- dir.patch
index 7feee0f..3ba4711 100644
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +1 -2
@@ -1,13 +1,14 @@
-Add -s <directory> support to populate the image.
+Add -d <directory> support to populate the image.
 
 Upstream-Status: Inappropriate [licensing]
 We're tracking an old release of dosfstools due to licensing issues.
 
 Signed-off-by: Scott Garman <[email protected]>
 
-diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c dosfstools-
2.10/mkdosfs/mkdosfs.c
---- dosfstools-2.10.orig/mkdosfs/mkdosfs.c 2004-08-02
20:48:45.000000000 -0700
-+++ dosfstools-2.10/mkdosfs/mkdosfs.c 2004-08-02 20:49:44.296953792
-0700
+Index: dosfstools-2.11/mkdosfs/mkdosfs.c
+===================================================================
+--- dosfstools-2.11.orig/mkdosfs/mkdosfs.c   2011-12-06
12:27:55.000000000 +0000
++++ dosfstools-2.11/mkdosfs/mkdosfs.c 2011-12-06 12:37:13.445950703
+0000
@@ -18,6 +18,10 @@
as a rule), and not the block. For example the boot block does
not
occupy a full cluster.
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -26,18 +27,18 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
+#include <libgen.h>
+#include <dirent.h>
- #if __BYTE_ORDER == __BIG_ENDIAN
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +4 -4
-
-@@ -124,6 +130,8 @@
- }
- #endif
+ #include <linux/version.h>
+ #if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
+@@ -110,6 +116,8 @@
+  * sufficient (or even better :) for 64 bit offsets in the meantime
*/
+ #define llseek lseek
+#define ROUND_UP(value, divisor) (value + (divisor - (value %
divisor))) / divisor
+
/* Constant definitions */
#define TRUE 1 /* Boolean constants */
-@@ -163,7 +171,6 @@
+@@ -149,7 +157,6 @@
  #define ATTR_VOLUME  8              /* volume label */
  #define ATTR_DIR     16             /* directory */
  #define ATTR_ARCH    32             /* archived */
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -45,7 +46,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
#define ATTR_NONE 0 /* no attribute bits */
#define ATTR_UNUSED (ATTR_VOLUME | ATTR_ARCH | ATTR_SYS |
ATTR_HIDDEN)
      /* attribute bits that are copied "as is" */
-@@ -258,6 +265,19 @@
+@@ -245,6 +252,19 @@
    __u32             reserved2[4];
};
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -65,7 +66,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
struct msdos_dir_entry
{
char name[8], ext[3]; /* name and extension */
-@@ -306,6 +326,15 @@
+@@ -293,6 +313,15 @@
  #define MESSAGE_OFFSET 29   /* Offset of message in above code */
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -81,7 +82,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
/* Global variables - the root of all evil :-) - see these and weep!
*/
static char *template_boot_code; /* Variable to store a full
template boot sector in */
-@@ -339,6 +368,9 @@
+@@ -326,6 +355,9 @@
static int size_root_dir; /* Size of the root directory in bytes */
static int sectors_per_cluster = 0; /* Number of sectors per disk
cluster */
static int root_dir_entries = 0; /* Number of root directory entries
*/
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -89,9 +90,9 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
+static int last_cluster_written = 0;
+
static char *blank_sector; /* Blank sector - all zeros */
+ static int hidden_sectors = 0; /* Number of hidden sectors
*/
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +1 -2
-
-@@ -411,7 +443,6 @@
+@@ -399,7 +431,6 @@
    }
  }
@@ -99,7 +100,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
/* Mark a specified sector as having a particular value in it's FAT
entry */
static void
-@@ -1262,6 +1293,9 @@
+@@ -1266,6 +1297,9 @@
        die ("unable to allocate space for root directory in memory");
      }
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -109,7 +110,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
    memset(root_dir, 0, size_root_dir);
    if ( memcmp(volume_name, "           ", 11) )
      {
-@@ -1310,11 +1344,11 @@
+@@ -1314,11 +1348,11 @@
}
if (!(blank_sector = malloc( sector_size )))
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -124,7 +125,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
/* Write the new filesystem's data tables to wherever they're going
to end up! */
#define error(str) \
-@@ -1336,7 +1370,7 @@
+@@ -1340,7 +1374,7 @@
    do {                                                      \
      int __size = (size);                            \
      if (write (dev, buf, __size) != __size)         \
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -133,7 +134,7 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
    } while(0)
-@@ -1407,6 +1441,452 @@
+@@ -1412,6 +1446,452 @@
free (fat); /* Free up the fat table space reserved during
setup_tables */
}
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +0 -0
@@ -586,19 +587,16 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
/* Report the command usage and return a failure error code */
-@@ -1418,9 +1898,9 @@
+@@ -1423,7 +1903,7 @@
         [-m boot-msg-file] [-n volume-name] [-i volume-id] [-B
bootcode]\n\
[-s sectors-per-cluster] [-S logical-sector-size] [-f number-
of-FATs]\n\
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +1 -1
-        [-F fat-size] [-r root-dir-entries] [-R reserved-sectors]\n\
+        [-h hidden-sectors] [-F fat-size] [-r root-dir-entries] [-R
reserved-sectors]\n\
- /dev/name [blocks]\n");
+ [-d directory] /dev/name [blocks]\n");
}
--
-+
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +1 -1
+
  /*
-  * ++roman: On m68k, check if this is an Atari; if yes, turn on Atari
variant
meta/recipes-devtools/dosfstools/dosfstools/mkdosfs-dir.patch +1 -2
-  * of MS-DOS filesystem by default.
-@@ -1458,6 +1938,8 @@
+@@ -1463,6 +1943,8 @@
    int c;
    char *tmp;
    char *listfile = NULL;
@@ -607,27 +605,27 @@ diff -urN dosfstools-2.10.orig/mkdosfs/mkdosfs.c
dosfstools-2.10/mkdosfs/mkdosfs
    FILE *msgfile;
    struct stat statbuf;
    int i = 0, pos, ch;
-@@ -1477,7 +1959,7 @@
+@@ -1483,7 +1965,7 @@
    printf ("%s " VERSION " (" VERSION_DATE ")\n",
         program_name);
-- while ((c = getopt (argc, argv, "AcCf:F:Ii:l:m:n:r:R:s:S:v:B:b")) != EOF)
-+ while ((c = getopt (argc, argv, "AcCd:f:F:Ii:l:m:n:r:R:s:S:v:B:b")) != EOF)
+- while ((c = getopt (argc, argv, "AbcCf:F:Ii:l:m:n:r:R:s:S:v:B:")) != EOF)
++ while ((c = getopt (argc, argv, "AbcCd:f:F:Ii:l:m:n:r:R:s:S:v:B:")) != EOF)
      /* Scan the command line for options */
      switch (c)
        {
-@@ -1502,6 +1984,10 @@
+@@ -1508,6 +1990,10 @@
      create = TRUE;
      break;
-+ case 'd':
++ case 'd':
+ dirname = optarg;
+ break;
+
        case 'f':             /* f : Choose number of FATs */
      nr_fats = (int) strtol (optarg, &tmp, 0);
      if (*tmp || nr_fats < 1 || nr_fats > 4)
-@@ -1796,8 +2282,10 @@
+@@ -1811,8 +2297,10 @@
else if (listfile)
get_list_blocks (listfile);
meta/recipes-devtools/dosfstools/dosfstools_2.10.bb deleted +0 -2
deleted file mode 100644
index c23c02d..0000000
@@ -1,24 +0,0 @@
-# dosfstools OE build file
-# Copyright (C) 2004-2006, Advanced Micro Devices, Inc.  All Rights
Reserved
meta/recipes-devtools/dosfstools/dosfstools_2.10.bb deleted +0 -7
-# Released under the MIT license (see packages/COPYING)
-
-DESCRIPTION = "DOS FAT Filesystem Utilities"
-
-SECTION = "base"
-LICENSE = "GPLv2"
-LIC_FILES_CHKSUM =
"file://mkdosfs/COPYING;md5=cbe67f08d6883bff587f615f0cc81aa8"
meta/recipes-devtools/dosfstools/dosfstools_2.10.bb deleted +0 -3
-PR = "r3"
-
-SRC_URI = "ftp://ftp.uni-
erlangen.de/pub/Linux/LOCAL/dosfstools/dosfstools-${PV}.src.tar.gz \
meta/recipes-devtools/dosfstools/dosfstools_2.10.bb deleted +0 -7
-        file://alignment_hack.patch \
-        file://dosfstools-2.10-kernel-2.6.patch \
-           file://msdos_fat12_undefined.patch \
-        file://include-linux-types.patch"
-
-SRC_URI[md5sum] = "59a02f311a891af8787c4c9e28c6b89b"
-SRC_URI[sha256sum] =
"55a7b2f5ea4506bde935ee3145573e1773427fc72283a36796c7c2cf861dd064"
meta/recipes-devtools/dosfstools/dosfstools_2.10.bb deleted +0 -5
-
-do_install () {
-     oe_runmake "PREFIX=${D}" "SBINDIR=${D}${sbindir}" \
-                "MANDIR=${D}${mandir}/man8" install
-}
meta/recipes-devtools/dosfstools/dosfstools_2.11.bb +0 -0
index 944d873..7d0080c 100644
@@ -7,12 +7,15 @@ DESCRIPTION = "DOS FAT Filesystem Utilities"
 SECTION = "base"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM =
"file://mkdosfs/COPYING;md5=cbe67f08d6883bff587f615f0cc81aa8"
meta/recipes-devtools/dosfstools/dosfstools_2.11.bb +1 -1
-PR = "r0"
+PR = "r1"
 
 SRC_URI = "ftp://ftp.uni-
erlangen.de/pub/Linux/LOCAL/dosfstools/dosfstools-${PV}.src.tar.gz \
meta/recipes-devtools/dosfstools/dosfstools_2.11.bb +5 -2
-        file://alignment_hack.patch \
+           file://mkdosfs-bootcode.patch \
+           file://mkdosfs-dir.patch \
+           file://alignment_hack.patch \
            file://msdos_fat12_undefined.patch \
-        file://include-linux-types.patch"
+           file://dosfstools-msdos_fs-types.patch \
+           file://include-linux-types.patch"
 
 SRC_URI[md5sum] = "407d405ade410f7597d364ab5dc8c9f6"
 SRC_URI[sha256sum] =
"0eac6d12388b3d9ed78684529c1b0d9346fa2abbe406c4d4a3eb5a023c98a484"
meta/recipes-devtools/dosfstools/dosfstools_2.11.bb +2 -1
@@ -21,3 +24,5 @@ do_install () {
      oe_runmake "PREFIX=${D}" "SBINDIR=${D}${sbindir}" \
                 "MANDIR=${D}${mandir}/man8" install
 }
+
+BBCLASSEXTEND = "native"
--
1.7.2.5
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Darren Hart

2011-12-09 19:21 UTC #6 lore raw

Re: [PATCH 2/3] dosfstools: update native to 2.11

In reply to Otavio Salvador

Quoted 25 lines · Otavio Salvador

This unify recipes for target and native builds and also drops the the already merged patches.

Signed-off-by Otavio Salvador <[email protected]>

---
 .../dosfstools/dosfstools-native_2.10.bb           |   19 -----
 .../dosfstools/dosfstools/2.6.20-syscall.patch     |   72 -------------------
 .../dosfstools/dosfstools-2.10-kernel-2.6.patch    |   74 --------------------
 .../dosfstools/dosfstools/mkdosfs-bootcode.patch   |   68 ++++++++----------
 .../dosfstools/dosfstools/mkdosfs-dir.patch        |   62 ++++++++---------
 .../recipes-devtools/dosfstools/dosfstools_2.10.bb |   24 ------
 .../recipes-devtools/dosfstools/dosfstools_2.11.bb |   11 ++-
 7 files changed, 69 insertions(+), 261 deletions(-)
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools_2.10.bb

meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch deleted +0 -0

deleted file mode 100644
index 4460f06..0000000

meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch line 4

Quoted 4 lines

meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch deleted +0 -4

@@ -1,72 +0,0 @@
-Only use the system's llseek().
-
-Upstream-Status: Inappropriate [licensing]
-We're tracking an old release of dosfstools due to licensing issues.

What are these licensing issues, and are they no longer an issue?

Signature · 3 lines

Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Otavio Salvador

author

2011-12-09 19:24 UTC #7 lore raw

Re: [PATCH 2/3] dosfstools: update native to 2.11

Quoted 2 lines · Darren Hart

What are these licensing issues, and are they no longer an issue?

Newer dosfstools is GPLv3.

Signature · 3 lines

Otavio Salvador                             O.S. Systems
E-mail: [email protected]  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Darren Hart

2011-12-09 19:27 UTC #8 lore raw

Re: [PATCH 2/3] dosfstools: update native to 2.11

Quoted 7 lines · Otavio Salvador

On Fri, Dec 9, 2011 at 17:15, Darren Hart <[email protected] <mailto:[email protected]>> wrote:
What are these licensing issues, and are they no longer an issue?
Newer dosfstools is GPLv3.

So pulling this in would mean we can't build live images when building non GPLv3 images? We are looking to revamp the image creation, but until then I can't agree to this.

Signature · 3 lines

Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Otavio Salvador

author

2011-12-09 19:42 UTC #9 lore raw

Re: [PATCH 2/3] dosfstools: update native to 2.11

Quoted 3 lines · Darren Hart

So pulling this in would mean we can't build live images when building non GPLv3 images? We are looking to revamp the image creation, but until then I can't agree to this.

NO. This is 2.11 release.

The GPLv3 versions are the 3.0.x (now it is at 3.0.12).

Signature · 3 lines

Otavio Salvador                             O.S. Systems
E-mail: [email protected]  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Darren Hart

2011-12-09 19:49 UTC #10 lore raw

Re: [PATCH 2/3] dosfstools: update native to 2.11

Quoted 11 lines · Otavio Salvador

On Fri, Dec 9, 2011 at 17:21, Darren Hart <[email protected] <mailto:[email protected]>> wrote:
So pulling this in would mean we can't build live images when building non GPLv3 images? We are looking to revamp the image creation, but until then I can't agree to this.
NO. This is 2.11 release.
The GPLv3 versions are the 3.0.x (now it is at 3.0.12).

Ah, this should be fine then. Otavio says the llseek patch which had the comment regarding licensing is in 2.11.

No objection your honor.

Signature · 3 lines

Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Saul Wold

Applied

2011-12-12 23:22 UTC #11 lore raw

Re: [PATCH 2/3] dosfstools: update native to 2.11

In reply to Otavio Salvador

Quoted 13 lines · Otavio Salvador

This unify recipes for target and native builds and also drops the the already merged patches.

Signed-off-by Otavio Salvador<[email protected]>

---
  .../dosfstools/dosfstools-native_2.10.bb           |   19 -----
  .../dosfstools/dosfstools/2.6.20-syscall.patch     |   72 -------------------
  .../dosfstools/dosfstools-2.10-kernel-2.6.patch    |   74 --------------------
  .../dosfstools/dosfstools/mkdosfs-bootcode.patch   |   68 ++++++++----------
  .../dosfstools/dosfstools/mkdosfs-dir.patch        |   62 ++++++++---------
  .../recipes-devtools/dosfstools/dosfstools_2.10.bb |   24 ------
  .../recipes-devtools/dosfstools/dosfstools_2.11.bb |   11 ++-
  7 files changed, 69 insertions(+), 261 deletions(-)

Quoted 4 lines

delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools_2.10.bb

Merged into OE-Core

Thanks
Sau!

Richard Purdie

2011-12-13 15:56 UTC #12 lore raw

Re: [PATCH 2/3] dosfstools: update native to 2.11

In reply to Otavio Salvador

Quoted 4 lines · Otavio Salvador

This unify recipes for target and native builds and also drops the the already merged patches.
Signed-off-by Otavio Salvador <[email protected]>

Quoted 13 lines

---
 .../dosfstools/dosfstools-native_2.10.bb           |   19 -----
 .../dosfstools/dosfstools/2.6.20-syscall.patch     |   72 -------------------
 .../dosfstools/dosfstools-2.10-kernel-2.6.patch    |   74 --------------------
 .../dosfstools/dosfstools/mkdosfs-bootcode.patch   |   68 ++++++++----------
 .../dosfstools/dosfstools/mkdosfs-dir.patch        |   62 ++++++++---------
 .../recipes-devtools/dosfstools/dosfstools_2.10.bb |   24 ------
 .../recipes-devtools/dosfstools/dosfstools_2.11.bb |   11 ++-
 7 files changed, 69 insertions(+), 261 deletions(-)
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools-native_2.10.bb
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/2.6.20-syscall.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools/dosfstools-2.10-kernel-2.6.patch
 delete mode 100644 meta/recipes-devtools/dosfstools/dosfstools_2.10.bb

This broke large live images:

http://autobuilder.yoctoproject.org:8010/builders/crownbay-noemgd/builds/39/steps/shell_30/logs/stdio

and many other failure reports. I've figured out what changed and pushed a fix:

http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=995a09e7bf3aee7853b17a7757cbf6cbc4edf6b8

Cheers,

Richard

Otavio Salvador

author

2011-12-08 15:47 UTC #13 lore raw

PATCH 3/3 scripts/send-pull-request: allow sending of patches without a cover letter

In reply to Otavio Salvador

Signed-off-by Otavio Salvador <[email protected]>

---
 scripts/send-pull-request |   21 ++++++++++++---------
 1 files changed, 12 insertions(+), 9 deletions(-)

scripts/send-pull-request +12 -9

index be13009..8d51ce2 100755
@@ -108,15 +108,18 @@ fi
 
 
 # Verify the cover letter is complete and free of tokens
-CL="$PDIR/0000-cover-letter.patch"
-for TOKEN in SUBJECT BLURB; do
-	grep -q "*** $TOKEN HERE ***" "$CL"
-	if [ $? -eq 0 ]; then
-		echo "ERROR: Please edit $CL and try again (Look for '*** $TOKEN HERE ***')."
-		exit 1
-	fi
-done
-
+if [ -e $PDIR/0000-cover-letter.patch ]; then
+	CL="$PDIR/0000-cover-letter.patch"
+	for TOKEN in SUBJECT BLURB; do
+		grep -q "*** $TOKEN HERE ***" "$CL"
+		if [ $? -eq 0 ]; then
+			echo "ERROR: Please edit $CL and try again (Look for '*** $TOKEN HERE ***')."
+			exit 1
+		fi
+	done
+else
+	echo "WARNING: No cover letter will be send."
+fi
 
 # Harvest emails from the generated patches and populate AUTO_CC.
 if [ $AUTO_CL -eq 1 ]; then

Signature · 1 line

1.7.2.5

Darren Hart

2011-12-09 19:25 UTC #14 lore raw

Re: [PATCH 3/3] scripts/send-pull-request: allow sending of patches without a cover letter

On 12/08/2011 07:40 AM, Otavio Salvador wrote:

Hi Otavio,

Please provide a commit log body.

http://wiki.openembedded.org/index.php/Commit_Patch_Message_Guidelines#Patch_Headers_and_Commit_Messages

scripts/send-pull-request fi · line 121

Quoted 2 lines

scripts/send-pull-request +2 -0
+else
+	echo "WARNING: No cover letter will be send."

s/send/sent/

Otherwise looks reasonable.

Signature · 3 lines

Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Richard Purdie

2011-12-10 00:13 UTC #15 lore raw

Re: [PATCH 3/3] scripts/send-pull-request: allow sending of patches without a cover letter

Quoted 9 lines · Darren Hart

On 12/08/2011 07:40 AM, Otavio Salvador wrote:
Hi Otavio,
Please provide a commit log body.
http://wiki.openembedded.org/index.php/Commit_Patch_Message_Guidelines#Patch_Headers_and_Commit_Messages

Quoted 7 lines

scripts/send-pull-request fi · line 121
Quoted 2 lines
scripts/send-pull-request +2 -0
+else
+	echo "WARNING: No cover letter will be send."
s/send/sent/
Otherwise looks reasonable.

Hmm, this has just merged with the typo, sorry (it had been around a while with no comments). Can someone sent a follow up please?

Cheers,

Richard

Otavio Salvador

author

2011-12-10 15:58 UTC #16 lore raw

Re: [PATCH 3/3] scripts/send-pull-request: allow sending of patches without a cover letter

Quoted 3 lines · Richard Purdie

Hmm, this has just merged with the typo, sorry (it had been around a while with no comments). Can someone sent a follow up please?

I will send a patch fixing it.

Signature · 3 lines

Otavio Salvador                             O.S. Systems
E-mail: [email protected]  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Khem Raj

2011-12-09 19:41 UTC #17 lore raw

Re: [PATCH 3/3] scripts/send-pull-request: allow sending of patches without a cover letter

In reply to Otavio Salvador

Quoted 36 lines · Otavio Salvador

Signed-off-by Otavio Salvador <[email protected]>

---
scripts/send-pull-request | 21 ++++++++++++---------

 1 files changed, 12 insertions(+), 9 deletions(-)

scripts/send-pull-request +12 -9

index be13009..8d51ce2 100755
@@ -108,15 +108,18 @@ fi
 
 
  # Verify the cover letter is complete and free of tokens
-CL="$PDIR/0000-cover-letter.patch"
-for TOKEN in SUBJECT BLURB; do
-       grep -q "*** $TOKEN HERE ***" "$CL"
-       if [ $? -eq 0 ]; then
-               echo "ERROR: Please edit $CL and try again (Look for '*** $TOKEN HERE ***')."
-               exit 1
-       fi
-done
-
+if [ -e $PDIR/0000-cover-letter.patch ]; then
+       CL="$PDIR/0000-cover-letter.patch"
+       for TOKEN in SUBJECT BLURB; do
+               grep -q "*** $TOKEN HERE ***" "$CL"
+               if [ $? -eq 0 ]; then
+                       echo "ERROR: Please edit $CL and try again (Look for '*** $TOKEN HERE ***')."
+                       exit 1
+               fi
+       done
+else
+       echo "WARNING: No cover letter will be send."
+fi
 
  # Harvest emails from the generated patches and populate AUTO_CC.

Quoted 5 lines

if [ $AUTO_CL -eq 1 ]; then
--
1.7.2.5

why do you want to avoid cover letter ? or is it for other layers other than oe-core

Otavio Salvador

author

2011-12-09 19:43 UTC #18 lore raw

Re: [PATCH 3/3] scripts/send-pull-request: allow sending of patches without a cover letter

Quoted 3 lines · Khem Raj

why do you want to avoid cover letter ? or is it for other layers other than oe-core

When it is a single patch makes no sense to send a cover letter that most of time we just write the same of first commit.

Signature · 3 lines

Otavio Salvador                             O.S. Systems
E-mail: [email protected]  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Darren Hart

2011-12-09 19:52 UTC #19 lore raw

Re: [PATCH 3/3] scripts/send-pull-request: allow sending of patches without a cover letter

Quoted 9 lines · Otavio Salvador

On Fri, Dec 9, 2011 at 17:34, Khem Raj <[email protected] <mailto:[email protected]>> wrote:
why do you want to avoid cover letter ? or is it for other layers other than oe-core
When it is a single patch makes no sense to send a cover letter that most of time we just write the same of first commit.

Well, you could just use git-send-email when there is only 1 patch. I had considered having the tool create only a single email if there was only 1 patch. However, in the end, I just send the cover letter with the pull requrest and let the single patch follow. It works fine and is consistent with the other patches, even if it is a degenerate case.

So I don't think this is necessary really, but if it works for you and doesn't get abused, I'm fine with it. Perhaps only allowing this if there is indeed only 1 patch would be better.

Signature · 3 lines

Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel

Khem Raj

2011-12-09 19:55 UTC #20 lore raw

Re: [PATCH 3/3] scripts/send-pull-request: allow sending of patches without a cover letter

Quoted 4 lines · Darren Hart

Quoted 3 lines · Otavio Salvador
On Fri, Dec 9, 2011 at 17:34, Khem Raj <[email protected] <mailto:[email protected]>> wrote:

Quoted 8 lines

Quoted 6 lines
why do you want to avoid cover letter ? or is it for other layers other than oe-core
When it is a single patch makes no sense to send a cover letter that most of time we just write the same of first commit.
Well, you could just use git-send-email when there is only 1 patch.

right

Quoted 8 lines

had considered having the tool create only a single email if there was only 1 patch. However, in the end, I just send the cover letter with the pull requrest and let the single patch follow. It works fine and is consistent with the other patches, even if it is a degenerate case.
So I don't think this is necessary really, but if it works for you and doesn't get abused, I'm fine with it. Perhaps only allowing this if there is indeed only 1 patch would be better.

if its possible it will happen abuse that is