Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] Add package libgeotiff
@ 2011-06-13 10:18 Murat Demirten
  2011-07-07 20:55 ` Peter Korsgaard
  0 siblings, 1 reply; 2+ messages in thread
From: Murat Demirten @ 2011-06-13 10:18 UTC (permalink / raw)
  To: buildroot

Hi,

Libgeotiff is an open source library normally hosted on top of libtiff for
reading, and writing GeoTIFF information tags.

Regards,

[master b7094a9] package: libgeotiff added.
 3 files changed, 26 insertions(+), 0 deletions(-)
 create mode 100644 package/libgeotiff/Config.in
 create mode 100644 package/libgeotiff/libgeotiff.mk

---
 package/Config.in                |    1 +
 package/libgeotiff/Config.in     |    7 +++++++
 package/libgeotiff/libgeotiff.mk |   18 ++++++++++++++++++
 3 files changed, 26 insertions(+), 0 deletions(-)
 create mode 100644 package/libgeotiff/Config.in
 create mode 100644 package/libgeotiff/libgeotiff.mk

diff --git a/package/Config.in b/package/Config.in
index 81701d8..7b43143 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -280,6 +280,7 @@ source "package/libart/Config.in"
 source "package/libdrm/Config.in"
 source "package/libexif/Config.in"
 source "package/libgail/Config.in"
+source "package/libgeotiff/Config.in"
 source "package/libglade/Config.in"
 source "package/gdk-pixbuf/Config.in"
 source "package/libgtk2/Config.in"
diff --git a/package/libgeotiff/Config.in b/package/libgeotiff/Config.in
new file mode 100644
index 0000000..47420fd
--- /dev/null
+++ b/package/libgeotiff/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_LIBGEOTIFF
+       bool "libgeotiff"
+       select BR2_PACKAGE_TIFF
+       help
+        Libgeotiff is an open source library normally hosted on top of
libtiff for reading, and writing GeoTIFF information tags.
+
+        http://geotiff.osgeo.org/
diff --git a/package/libgeotiff/libgeotiff.mk b/package/libgeotiff/
libgeotiff.mk
new file mode 100644
index 0000000..742fd75
--- /dev/null
+++ b/package/libgeotiff/libgeotiff.mk
@@ -0,0 +1,18 @@
+#############################################################
+#
+# libgeotiff
+#
+#############################################################
+LIBGEOTIFF_VERSION = 1.3.0
+LIBGEOTIFF_SOURCE = libgeotiff-$(LIBGEOTIFF_VERSION).tar.gz
+LIBGEOTIFF_SITE = http://download.osgeo.org/geotiff/libgeotiff
+LIBGEOTIFF_DEPENDENCIES = tiff
+LIBGEOTIFF_INSTALL_STAGING = YES
+LIBGEOTIFF_INSTALL_TARGET = YES
+
+define LIBGEOTIFF_INSTALL_TARGET_CMDS
+       mkdir -p $(TARGET_DIR)/usr/lib
+       cp -dpf $(@D)/.libs/libgeotiff.so* $(TARGET_DIR)/usr/lib/
+endef
+
+$(eval $(call AUTOTARGETS,package,libgeotiff))
-- 
1.7.5.4
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20110613/3df2c59a/attachment.html>

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH] Add package libgeotiff
  2011-06-13 10:18 [Buildroot] [PATCH] Add package libgeotiff Murat Demirten
@ 2011-07-07 20:55 ` Peter Korsgaard
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Korsgaard @ 2011-07-07 20:55 UTC (permalink / raw)
  To: buildroot

>>>>> "Murat" == Murat Demirten <mdemirten@yh.com.tr> writes:

 Murat> Hi,
 Murat> Libgeotiff is an open source library normally hosted on top of libtiff for
 Murat> reading, and writing GeoTIFF information tags.

 Murat> Regards,

Thanks, a few comments:

 Murat> [master b7094a9] package: libgeotiff added.
 Murat> ?3 files changed, 26 insertions(+), 0 deletions(-)
 Murat> ?create mode 100644 package/libgeotiff/Config.in
 Murat> ?create mode 100644 package/libgeotiff/libgeotiff.mk

Your mailer seems to corrupt the patch, so I cannot apply it with git
am. Please consider sending patches with git send-email in the future.

 Murat> ---
 Murat> ?package/Config.in ? ? ? ? ? ? ? ?| ? ?1 +
 Murat> ?package/libgeotiff/Config.in ? ? | ? ?7 +++++++
 Murat> ?package/libgeotiff/libgeotiff.mk | ? 18 ++++++++++++++++++
 Murat> ?3 files changed, 26 insertions(+), 0 deletions(-)
 Murat> ?create mode 100644 package/libgeotiff/Config.in
 Murat> ?create mode 100644 package/libgeotiff/libgeotiff.mk

 Murat> diff --git a/package/Config.in b/package/Config.in
 Murat> index 81701d8..7b43143 100644
 Murat> --- a/package/Config.in
 Murat> +++ b/package/Config.in
 Murat> @@ -280,6 +280,7 @@ source "package/libart/Config.in"
 Murat> ?source "package/libdrm/Config.in"
 Murat> ?source "package/libexif/Config.in"
 Murat> ?source "package/libgail/Config.in"
 Murat> +source "package/libgeotiff/Config.in"
 Murat> ?source "package/libglade/Config.in"
 Murat> ?source "package/gdk-pixbuf/Config.in"
 Murat> ?source "package/libgtk2/Config.in"
 Murat> diff --git a/package/libgeotiff/Config.in b/package/libgeotiff/Config.in
 Murat> new file mode 100644
 Murat> index 0000000..47420fd
 Murat> --- /dev/null
 Murat> +++ b/package/libgeotiff/Config.in
 Murat> @@ -0,0 +1,7 @@
 Murat> +config BR2_PACKAGE_LIBGEOTIFF
 Murat> + ? ? ? bool "libgeotiff"
 Murat> + ? ? ? select BR2_PACKAGE_TIFF
 Murat> + ? ? ? help
 Murat> + ? ? ? ?Libgeotiff is an open source library normally hosted on top of libtiff
 Murat> for reading, and writing GeoTIFF information tags.?

Please keep help text lines < 70 chars.

 Murat> +
 Murat> + ? ? ? ?http://geotiff.osgeo.org/
 Murat> diff --git a/package/libgeotiff/libgeotiff.mk b/package/libgeotiff/
 Murat> libgeotiff.mk
 Murat> new file mode 100644
 Murat> index 0000000..742fd75
 Murat> --- /dev/null
 Murat> +++ b/package/libgeotiff/libgeotiff.mk
 Murat> @@ -0,0 +1,18 @@
 Murat> +#############################################################
 Murat> +#
 Murat> +# libgeotiff
 Murat> +#
 Murat> +#############################################################
 Murat> +LIBGEOTIFF_VERSION = 1.3.0
 Murat> +LIBGEOTIFF_SOURCE = libgeotiff-$(LIBGEOTIFF_VERSION).tar.gz
 Murat> +LIBGEOTIFF_SITE = http://download.osgeo.org/geotiff/libgeotiff
 Murat> +LIBGEOTIFF_DEPENDENCIES = tiff
 Murat> +LIBGEOTIFF_INSTALL_STAGING = YES
 Murat> +LIBGEOTIFF_INSTALL_TARGET = YES

Minor detail, the _SOURCE and INSTALL_TARGET lines can be removed as
this is their default value.

 Murat> +
 Murat> +define LIBGEOTIFF_INSTALL_TARGET_CMDS
 Murat> + ? ? ? mkdir -p $(TARGET_DIR)/usr/lib
 Murat> + ? ? ? cp -dpf $(@D)/.libs/libgeotiff.so* $(TARGET_DIR)/usr/lib/
 Murat> +endef

Why not just use make install like you do for the staging install.

 Murat> +
 Murat> +$(eval $(call AUTOTARGETS,package,libgeotiff))

Other than that it looks fine. Please fix these minor issues and resend
(with git send-email or as an attachment if that isn't possible).

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-07-07 20:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-13 10:18 [Buildroot] [PATCH] Add package libgeotiff Murat Demirten
2011-07-07 20:55 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox