Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Source tarball 1.2.1 contains slight differences to the files in the repository #4

Open
GoogleCodeExporter opened this issue Jul 6, 2015 · 1 comment

Comments

@GoogleCodeExporter
Copy link

There are slight discrepancies between the contents of the tarball and the
contents of the svn repository.

The release process should be amended such that the tarballs exactly
reflect the repository state.

See also issue 3.

diff -rux.svn google-coredumper/Makefile.am coredumper-1.2.1/Makefile.am
--- google-coredumper/Makefile.am   2008-10-01 15:05:28.000000000 +0100
+++ coredumper-1.2.1/Makefile.am    2008-04-02 00:37:59.000000000 +0100
@@ -1,5 +1,8 @@
 ## Process this file with automake to produce Makefile.in

+# Make sure that when we re-make ./configure, we get the macros we need
+ACLOCAL_AMFLAGS = -I `pwd`/../autoconf
+
 # This is so we can #include <google/foo>
 AM_CPPFLAGS = -I$(top_srcdir)/src

diff -rux.svn google-coredumper/Makefile.in coredumper-1.2.1/Makefile.in
--- google-coredumper/Makefile.in   2008-10-01 15:05:28.000000000 +0100
+++ coredumper-1.2.1/Makefile.in    2008-04-04 16:24:29.000000000 +0100
@@ -227,6 +227,9 @@
 sysconfdir = @sysconfdir@
 target_alias = @target_alias@

+# Make sure that when we re-make ./configure, we get the macros we need
+ACLOCAL_AMFLAGS = -I `pwd`/../autoconf
+
 # This is so we can #include <google/foo>
 AM_CPPFLAGS = -I$(top_srcdir)/src
 googleincludedir = $(includedir)/google
Only in google-coredumper/: svn-commit.tmp

Original issue reported on code.google.com by m...@j.maxb.eu on 1 Oct 2008 at 2:19

@GoogleCodeExporter
Copy link
Author

Another difference:

diff -p -r coredumper-1.2.1/src/linux_syscall_support_unittest.cc 
google-coredumper-read-only/src/linux_syscall_support_unittest.cc
*** coredumper-1.2.1/src/linux_syscall_support_unittest.cc  2008-03-29 
06:38:57.000000000 +0800
--- 
google-coredumper-read-only/src/linux_syscall_support_unittest.cc   2012-08-29 
14:45:36.889474958 +0800
*************** namespace linux_syscall_support {
*** 62,68 ****
  #include <asm/posix_types.h>
  #include <asm/types.h>
  #include <errno.h>
- #include <linux/dirent.h>
  #include <linux/types.h>
  #include <linux/unistd.h>
  #include <signal.h>
--- 62,67 ----
*************** static void CheckStructures() {
*** 91,98 ****
    // there are a small number of data structures (e.g "struct
    // kernel_old_sigaction") that we cannot test at all, as glibc does
    // not have any definitions for them.
-   CHECK(sizeof(struct dirent64)  == sizeof(struct kernel_dirent64));
-   CHECK(sizeof(struct dirent)    == sizeof(struct kernel_dirent));
    CHECK(sizeof(struct iovec)     == sizeof(struct kernel_iovec));
    CHECK(sizeof(struct msghdr)    == sizeof(struct kernel_msghdr));
    CHECK(sizeof(struct pollfd)    == sizeof(struct kernel_pollfd));
--- 90,95 ----
diff -p -r coredumper-1.2.1/src/linuxthreads.c 
google-coredumper-read-only/src/linuxthreads.c
*** coredumper-1.2.1/src/linuxthreads.c 2008-03-29 06:38:57.000000000 +0800
--- google-coredumper-read-only/src/linuxthreads.c  2012-08-29 
14:45:36.893469332 +0800
*************** extern "C" {
*** 49,55 ****
  #include <asm/fcntl.h>
  #include <asm/posix_types.h>
  #include <asm/types.h>
- #include <linux/dirent.h>

  #include "linux_syscall_support.h"
  #include "thread_lister.h"
--- 49,54 ----

Original comment by rainco...@gmail.com on 29 Aug 2012 at 6:50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant