[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-09-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380

David Tardon  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE
   Assignee|ignate...@redhat.com|dtar...@redhat.com
Last Closed||2016-09-08 09:58:29



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-09-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380



--- Comment #8 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/libzmf

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-09-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380



--- Comment #7 from Igor Gnatenko  ---
(In reply to Ralf Corsepius from comment #6)
> @Igor, all these macros do is to obscure what they actually do. This is
> featuritis and serves no technical purpose.
> 
> Enforcing them is infantile bureaucracy.

will you force people to remember what is %{_make_output_sync} and all other
macro which could be added in future? Answering why we need macro for this -
EL6/EL7 GNU Make doesn't support "-O", but 4.0+ does.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-09-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380



--- Comment #6 from Ralf Corsepius  ---
@Igor, all these macros do is to obscure what they actually do. This is
featuritis and serves no technical purpose.

Enforcing them is infantile bureaucracy.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-09-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380



--- Comment #5 from Igor Gnatenko  ---
#--
# Tested features of make
# Output syncronization for parallel make:
%_make_output_sync %(! %{__make} --version -O >/dev/null 2>&1 || echo -O)

#--
# The "make" analogue, hiding the _smp_mflags magic from specs
%make_build %{__make} %{_make_output_sync} %{?_smp_mflags}


Basically %make_build is not anymore same as make %{?_smp_mflags} and I will
work with FPC to enforce such macro.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-09-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380

Ralf Corsepius  changed:

   What|Removed |Added

 CC||rc040...@freenet.de



--- Comment #4 from Ralf Corsepius  ---
(In reply to Igor Gnatenko from comment #3)
> Looks like I completely forgot about this, sorry.
> 
> > make %{?_smp_mflags}
> %make_build
> 
> > make install DESTDIR=%{buildroot}
> %make_install

Igor, enforcing these macros is not helpful. They are not mandatory.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-09-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380

Igor Gnatenko  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags||fedora-review+



--- Comment #3 from Igor Gnatenko  ---
Looks like I completely forgot about this, sorry.

> make %{?_smp_mflags}
%make_build

> make install DESTDIR=%{buildroot}
%make_install

* Missing BuildRequires: gcc-c++

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-09-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380



--- Comment #2 from David Tardon  ---
Spec URL: https://dtardon.fedorapeople.org/rpm/libzmf.spec
SRPM URL: https://dtardon.fedorapeople.org/rpm/libzmf-0.0.1-1.fc24.src.rpm
Copr builds:
https://copr.fedorainfracloud.org/coprs/dtardon/pending-review/build/450316/

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-07-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||libzmf



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1349380] Review Request: libzmf - a library for import of Zoner document formats

2016-06-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1349380

David Tardon  changed:

   What|Removed |Added

 CC||dtar...@redhat.com



--- Comment #1 from David Tardon  ---
Copr builds:
https://copr.fedoraproject.org/coprs/dtardon/pending-review/build/361498/

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org