* [Buildroot] [PATCH] smbnetfs: new package 0.5.3a @ 2013-12-02 23:09 Andrew Ruder 2013-12-03 8:47 ` Thomas Petazzoni 0 siblings, 1 reply; 5+ messages in thread From: Andrew Ruder @ 2013-12-02 23:09 UTC (permalink / raw) To: buildroot Signed-off-by: Andrew Ruder <andrew.ruder@elecsyscorp.com> --- package/Config.in | 1 + package/smbnetfs/Config.in | 16 ++++++++++++++++ package/smbnetfs/smbnetfs-compile-with-uclibc.patch | 14 ++++++++++++++ package/smbnetfs/smbnetfs.mk | 11 +++++++++++ 4 files changed, 42 insertions(+) create mode 100644 package/smbnetfs/Config.in create mode 100644 package/smbnetfs/smbnetfs-compile-with-uclibc.patch create mode 100644 package/smbnetfs/smbnetfs.mk diff --git a/package/Config.in b/package/Config.in index 6fb063d..4854383 100644 --- a/package/Config.in +++ b/package/Config.in @@ -148,6 +148,7 @@ source "package/makedevs/Config.in" source "package/mtd/Config.in" source "package/nfs-utils/Config.in" source "package/ntfs-3g/Config.in" +source "package/smbnetfs/Config.in" source "package/squashfs/Config.in" source "package/squashfs3/Config.in" source "package/sshfs/Config.in" diff --git a/package/smbnetfs/Config.in b/package/smbnetfs/Config.in new file mode 100644 index 0000000..fdc8167 --- /dev/null +++ b/package/smbnetfs/Config.in @@ -0,0 +1,16 @@ +config BR2_PACKAGE_SMBNETFS + bool "smbnetfs" + depends on BR2_PACKAGE_SAMBA + select BR2_PACKAGE_SAMBA_LIBSMBCLIENT + select BR2_PACKAGE_LIBFUSE + select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE + help + A user-space filesystem that contains an entire + SMB/NMB network under a single mount point. + Workgroups, servers, and shares can be browsed much + like the Network Neighborhood in Microsoft Window. + + http://smbnetfs.sourceforge.net/ + +comment "smbnetfs needs samba" + depends on !BR2_PACKAGE_SAMBA diff --git a/package/smbnetfs/smbnetfs-compile-with-uclibc.patch b/package/smbnetfs/smbnetfs-compile-with-uclibc.patch new file mode 100644 index 0000000..e5e9065 --- /dev/null +++ b/package/smbnetfs/smbnetfs-compile-with-uclibc.patch @@ -0,0 +1,14 @@ +Index: smbnetfs-0.5.3a/src/common.c +=================================================================== +--- smbnetfs-0.5.3a.orig/src/common.c 2013-11-15 11:22:27.520088835 -0600 ++++ smbnetfs-0.5.3a/src/common.c 2013-11-15 11:22:37.384088834 -0600 +@@ -4,7 +4,9 @@ + #include <stdarg.h> + #include <string.h> + #include <pthread.h> ++#ifdef HAVE_BACKTRACE + #include <execinfo.h> ++#endif + + #include "common.h" + diff --git a/package/smbnetfs/smbnetfs.mk b/package/smbnetfs/smbnetfs.mk new file mode 100644 index 0000000..1e01bd8 --- /dev/null +++ b/package/smbnetfs/smbnetfs.mk @@ -0,0 +1,11 @@ +################################################################################ +# +# smbnetfs +# +################################################################################ + +SMBNETFS_VERSION = 0.5.3a +SMBNETFS_SOURCE = smbnetfs-$(SMBNETFS_VERSION).tar.bz2 +SMBNETFS_SITE = http://downloads.sourceforge.net/smbnetfs + +$(eval $(autotools-package)) -- 1.8.4.rc3 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] smbnetfs: new package 0.5.3a 2013-12-02 23:09 [Buildroot] [PATCH] smbnetfs: new package 0.5.3a Andrew Ruder @ 2013-12-03 8:47 ` Thomas Petazzoni 2013-12-03 13:30 ` Andrew Ruder 0 siblings, 1 reply; 5+ messages in thread From: Thomas Petazzoni @ 2013-12-03 8:47 UTC (permalink / raw) To: buildroot Dear Andrew Ruder, Thanks for this contribution as well! On Mon, 2 Dec 2013 17:09:03 -0600, Andrew Ruder wrote: > Signed-off-by: Andrew Ruder <andrew.ruder@elecsyscorp.com> > --- > package/Config.in | 1 + > package/smbnetfs/Config.in | 16 ++++++++++++++++ > package/smbnetfs/smbnetfs-compile-with-uclibc.patch | 14 ++++++++++++++ We now normally wants patches to carry a sequence number, even if there is only one patch. So something like: smbnetfs-01-compile-with-uclibc.patch for example. See http://buildroot.org/downloads/manual/manual.html#patch-policy for details. > package/smbnetfs/smbnetfs.mk | 11 +++++++++++ > 4 files changed, 42 insertions(+) > create mode 100644 package/smbnetfs/Config.in > create mode 100644 package/smbnetfs/smbnetfs-compile-with-uclibc.patch > create mode 100644 package/smbnetfs/smbnetfs.mk > > diff --git a/package/Config.in b/package/Config.in > index 6fb063d..4854383 100644 > --- a/package/Config.in > +++ b/package/Config.in > @@ -148,6 +148,7 @@ source "package/makedevs/Config.in" > source "package/mtd/Config.in" > source "package/nfs-utils/Config.in" > source "package/ntfs-3g/Config.in" > +source "package/smbnetfs/Config.in" > source "package/squashfs/Config.in" > source "package/squashfs3/Config.in" > source "package/sshfs/Config.in" > diff --git a/package/smbnetfs/Config.in b/package/smbnetfs/Config.in > new file mode 100644 > index 0000000..fdc8167 > --- /dev/null > +++ b/package/smbnetfs/Config.in > @@ -0,0 +1,16 @@ > +config BR2_PACKAGE_SMBNETFS > + bool "smbnetfs" > + depends on BR2_PACKAGE_SAMBA > + select BR2_PACKAGE_SAMBA_LIBSMBCLIENT > + select BR2_PACKAGE_LIBFUSE > + select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE > + help > + A user-space filesystem that contains an entire > + SMB/NMB network under a single mount point. > + Workgroups, servers, and shares can be browsed much > + like the Network Neighborhood in Microsoft Window. > + > + http://smbnetfs.sourceforge.net/ Indentation of the help text is wrong. It should be one tab + two spaces. > +comment "smbnetfs needs samba" > + depends on !BR2_PACKAGE_SAMBA I'm not sure here if we want to depend on Samba, or select Samba. Usually, we prefer "select" so that things are transparent for the user. However, when the thing being selected is "big", and fairly obvious for the user (like the Python interpreter being needed to build a Python module), we use depends on. I'm not sure which choice to make here. However, what is sure is that you forgot to inherit the toolchain dependencies of libfuse: they should be added. > diff --git a/package/smbnetfs/smbnetfs-compile-with-uclibc.patch b/package/smbnetfs/smbnetfs-compile-with-uclibc.patch > new file mode 100644 > index 0000000..e5e9065 > --- /dev/null > +++ b/package/smbnetfs/smbnetfs-compile-with-uclibc.patch > @@ -0,0 +1,14 @@ All patches should have a description + Signed-off-by. Also, it would be good if you could submit that patch upstream, so that it gets integrated in a future release. > +Index: smbnetfs-0.5.3a/src/common.c > +=================================================================== > +--- smbnetfs-0.5.3a.orig/src/common.c 2013-11-15 11:22:27.520088835 -0600 > ++++ smbnetfs-0.5.3a/src/common.c 2013-11-15 11:22:37.384088834 -0600 > +@@ -4,7 +4,9 @@ > + #include <stdarg.h> > + #include <string.h> > + #include <pthread.h> > ++#ifdef HAVE_BACKTRACE > + #include <execinfo.h> > ++#endif > + > + #include "common.h" > + > diff --git a/package/smbnetfs/smbnetfs.mk b/package/smbnetfs/smbnetfs.mk > new file mode 100644 > index 0000000..1e01bd8 > --- /dev/null > +++ b/package/smbnetfs/smbnetfs.mk > @@ -0,0 +1,11 @@ > +################################################################################ > +# > +# smbnetfs > +# > +################################################################################ > + > +SMBNETFS_VERSION = 0.5.3a > +SMBNETFS_SOURCE = smbnetfs-$(SMBNETFS_VERSION).tar.bz2 > +SMBNETFS_SITE = http://downloads.sourceforge.net/smbnetfs > + > +$(eval $(autotools-package)) No SMBNETFS_DEPENDENCIES to ensure libfuse, libiconv and samba are built before smbnetfs? Also, we now request all new packages to have SMBNETFS_LICENSE and SMBNETFS_LICENSE_FILES. Thanks a lot! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] smbnetfs: new package 0.5.3a 2013-12-03 8:47 ` Thomas Petazzoni @ 2013-12-03 13:30 ` Andrew Ruder 2013-12-03 13:41 ` Thomas Petazzoni 0 siblings, 1 reply; 5+ messages in thread From: Andrew Ruder @ 2013-12-03 13:30 UTC (permalink / raw) To: buildroot Summary: I apologize for the shoddiness of this and my other patch and I will take the info you've given me here and make much better v2's :). On Tue, Dec 03, 2013 at 09:47:01AM +0100, Thomas Petazzoni wrote: > We now normally wants patches to carry a sequence number, even if there > is only one patch. So something like: > > smbnetfs-01-compile-with-uclibc.patch Ok, can do for v2. > Indentation of the help text is wrong. It should be one tab + two > spaces. I'll fix that for v2. > I'm not sure here if we want to depend on Samba, or select Samba. > Usually, we prefer "select" so that things are transparent for the > user. However, when the thing being selected is "big", and fairly > obvious for the user (like the Python interpreter being needed to build > a Python module), we use depends on. I'm not sure which choice to make > here. Yes, I had made the decision that samba was "intrusive" enough to require explicit selection. I figured since it required something outside of the Libraries section, I would make the user explicitely add it. > However, what is sure is that you forgot to inherit the toolchain > dependencies of libfuse: they should be added. Ah, I am a little unfamiliar with the intricacies of Kconfig and didn't realize I needed to explicitely do that. Will take care of it in v2. > All patches should have a description + Signed-off-by. Also, it would > be good if you could submit that patch upstream, so that it gets > integrated in a future release. OK, fair enough - can do on both counts. > No SMBNETFS_DEPENDENCIES to ensure libfuse, libiconv and samba are > built before smbnetfs? > > Also, we now request all new packages to have SMBNETFS_LICENSE and > SMBNETFS_LICENSE_FILES. v2 and v2. ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] smbnetfs: new package 0.5.3a 2013-12-03 13:30 ` Andrew Ruder @ 2013-12-03 13:41 ` Thomas Petazzoni 2013-12-03 13:46 ` Thomas De Schampheleire 0 siblings, 1 reply; 5+ messages in thread From: Thomas Petazzoni @ 2013-12-03 13:41 UTC (permalink / raw) To: buildroot Dear Andrew Ruder, On Tue, 3 Dec 2013 07:30:43 -0600, Andrew Ruder wrote: > Summary: I apologize for the shoddiness of this and my other patch and > I will take the info you've given me here and make much better > v2's :). Your v1 was really good! Only a few minor things need to be fixed, but overall, it's far from being shoddy, on the contrary! > > I'm not sure here if we want to depend on Samba, or select Samba. > > Usually, we prefer "select" so that things are transparent for the > > user. However, when the thing being selected is "big", and fairly > > obvious for the user (like the Python interpreter being needed to > > build a Python module), we use depends on. I'm not sure which > > choice to make here. > > Yes, I had made the decision that samba was "intrusive" enough to > require explicit selection. I figured since it required something > outside of the Libraries section, I would make the user explicitely > add it. I'm fine with this choice. We'll see what the other Buildroot developers think about this. > > However, what is sure is that you forgot to inherit the toolchain > > dependencies of libfuse: they should be added. > > Ah, I am a little unfamiliar with the intricacies of Kconfig and > didn't realize I needed to explicitely do that. Will take care of it > in v2. See http://buildroot.org/downloads/manual/manual.html#dependencies-target-toolchain-options for some details about this. But it's true that this part of the documentation doesn't really explain that the toolchain dependencies need to be propagated to all the reverse dependencies of a given package. The reason is that when a Config.in option A "selects Config.in option B, then the unmet dependencies of option B are ignored by the "select" made by option A. So we have to replicate the dependencies. > > All patches should have a description + Signed-off-by. Also, it > > would be good if you could submit that patch upstream, so that it > > gets integrated in a future release. > > OK, fair enough - can do on both counts. > > > No SMBNETFS_DEPENDENCIES to ensure libfuse, libiconv and samba are > > built before smbnetfs? > > > > Also, we now request all new packages to have SMBNETFS_LICENSE and > > SMBNETFS_LICENSE_FILES. > > v2 and v2. Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] smbnetfs: new package 0.5.3a 2013-12-03 13:41 ` Thomas Petazzoni @ 2013-12-03 13:46 ` Thomas De Schampheleire 0 siblings, 0 replies; 5+ messages in thread From: Thomas De Schampheleire @ 2013-12-03 13:46 UTC (permalink / raw) To: buildroot Hi, On Tue, Dec 3, 2013 at 2:41 PM, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote: [..] > >> > I'm not sure here if we want to depend on Samba, or select Samba. >> > Usually, we prefer "select" so that things are transparent for the >> > user. However, when the thing being selected is "big", and fairly >> > obvious for the user (like the Python interpreter being needed to >> > build a Python module), we use depends on. I'm not sure which >> > choice to make here. >> >> Yes, I had made the decision that samba was "intrusive" enough to >> require explicit selection. I figured since it required something >> outside of the Libraries section, I would make the user explicitely >> add it. > > I'm fine with this choice. We'll see what the other Buildroot > developers think about this. > I'm fine with either choice. However, if a 'depends on' is chosen, I don't think there should be a comment about it. We typically only add comments for toolchain dependencies. Best regards, Thomas ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-12-03 13:46 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-12-02 23:09 [Buildroot] [PATCH] smbnetfs: new package 0.5.3a Andrew Ruder 2013-12-03 8:47 ` Thomas Petazzoni 2013-12-03 13:30 ` Andrew Ruder 2013-12-03 13:41 ` Thomas Petazzoni 2013-12-03 13:46 ` Thomas De Schampheleire
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox