> @@ -230,9 +230,9 @@ rpmvar_DATA =
>
> install-exec-hook:
> @rm -f $(DESTDIR)$(bindir)/rpmquery
> - @LN_S@ ../../bin/rpm $(DESTDIR)$(bindir)/rpmquery
> + @LN_S@ rpm $(DESTDIR)$(bindir)/rpmquery
I would suggest making a new pull request that adds a configure switch for
changing
> @@ -86,20 +90,7 @@ char * stpncpy(char * dest, const char * src, size_t n);
> #define xstrdup(_str) rstrdup((_str))
> #define _free(_ptr) rfree((_ptr))
>
> -/* Retrofit glibc __progname */
> -#if defined __GLIBC__ && __GLIBC__ >= 2
> -#if __GLIBC_MINOR__ >= 1
> -#define __progname
> @@ -230,9 +230,9 @@ rpmvar_DATA =
>
> install-exec-hook:
> @rm -f $(DESTDIR)$(bindir)/rpmquery
> - @LN_S@ ../../bin/rpm $(DESTDIR)$(bindir)/rpmquery
> + @LN_S@ rpm $(DESTDIR)$(bindir)/rpmquery
The prefix is in general user-configurable. On my system, it's
`/usr/local/bin/`,
Closed #57.
---
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/57#event-546952766___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint
> @@ -560,6 +560,7 @@ dnl Checks for library functions.
> AC_CHECK_FUNCS(putenv)
> AC_CHECK_FUNCS(mempcpy)
> AC_CHECK_FUNCS(fdatasync)
> +AC_CHECK_DECLS(fdatasync, [], [], [#include ])
`fdatasync()` was just replaced with `fsync()` in
6151ac9a298a9fe560cf8f899dc0b0e67453446c.
---
Reply to
> @@ -230,9 +230,9 @@ rpmvar_DATA =
>
> install-exec-hook:
> @rm -f $(DESTDIR)$(bindir)/rpmquery
> - @LN_S@ ../../bin/rpm $(DESTDIR)$(bindir)/rpmquery
> + @LN_S@ rpm $(DESTDIR)$(bindir)/rpmquery
This is a problem here. I have no idea where "rpm" exists in this symlink
creation
Here is a set of patches to make rpm build cleanly on OS X.
Patches 1, 2, 3 should be pretty straightforward and safe.
Patch 4 looks like an outright bug. Maybe it works on other platforms because
the rpm packaging of rpm fixes it up somehow.
Patch 5 is more a hack to get it working, not
It's totally fine to submit this as a full pull request. If necessary, the
maintainers of the project can cherry pick from a pull request. It makes life
easier, in my opinion, to have them all in one pull request, anyway.
---
Reply to this email directly or view it on GitHub:
