[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309

Fedora Update System  changed:

   What|Removed |Added

 Resolution|RAWHIDE |ERRATA



--- Comment #14 from Fedora Update System  ---
gnome-latex-3.32.0-2.fc30, tepl-4.2.0-1.fc30 has been pushed to the Fedora 30
stable repository. If problems still persist, please make note of it in this
bug report.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #13 from Fedora Update System  ---
gnome-latex-3.32.0-2.fc30, tepl-4.2.0-1.fc30 has been pushed to the Fedora 30
testing repository. If problems still persist, please make note of it in this
bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-8aeaddc281

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #12 from Fedora Update System  ---
tepl-4.2.0-1.fc30 gnome-latex-3.32.0-2.fc30 has been submitted as an update to
Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-8aeaddc281

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309

Pete Walter  changed:

   What|Removed |Added

 Status|POST|CLOSED
   Fixed In Version||gnome-latex-3.32.0-2.fc31
 Resolution|--- |RAWHIDE
Last Closed||2019-03-16 10:14:17



-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #11 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/gnome-latex

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #10 from Pete Walter  ---
Thanks guys for the review! I've requested the repo now.

Help is always welcome! Thanks.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #9 from Phil Wyett  ---
(In reply to Robert-André Mauchin from comment #8)
> (In reply to Phil Wyett from comment #6)
> > If you use my original or latest spec file. You will not get the error.
> 
> I know, sorry but I need the original reporter to update his SPEC.
> Maybe you can work together to co-maitain this package? It would be great.
> 
>

Always happy to contribute.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #8 from Robert-André Mauchin  ---
(In reply to Phil Wyett from comment #6)
> If you use my original or latest spec file. You will not get the error.

I know, sorry but I need the original reporter to update his SPEC.
Maybe you can work together to co-maitain this package? It would be great.


 - Only issue: 

gnome-latex.src:53: W: mixed-use-of-spaces-and-tabs (spaces: line 53, tab: line
1)

  You need to either use spaces or tabs, not both.


Package is approved. Please work with Kalev and Phil's PR to update TEPL to 4.2
Please fix the above issue before import.



Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated", "FSF All Permissive License", "GPL (v3
 or later)", "Expat License", "GNU General Public License", "FSF
 Unlimited License (with Retention) GNU General Public License", "FSF
 Unlimited License (with Retention)", "GNU Lesser General Public
 License (v2.1 or later)", "GPL (v2 or later)". 1048 files have unknown
 license. Detailed output of licensecheck in
 /home/bob/packaging/review/gnome-latex/review-gnome-
 latex/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by: /usr/share/gtk-doc(vips-
 devel, iio-sensor-proxy-docs, garcon-devel, libxml2-devel,
 libcanberra-devel, gedit-devel, gtef-devel, libunity-gtk-parser-devel,
 gtk4-devel-docs, gtkspell3-devel, exo-devel, libgnome-devel,
 unique3-docs, gsequencer-devel-doc, gspell-doc, libnma-devel, gnome-
 online-accounts-devel, gnome-autoar-devel, libdbusmenu-doc, flatpak-
 devel, ibus-devel-docs, raptor2-devel, libmodulemd1-devel, anjuta-
 devel, clutter-gst3-devel, libgexiv2-devel, clipsmm-doc, libxfce4ui-
 devel, nautilus-devel, gtk-doc, xfconf-devel, gfbgraph-devel, uprof-
 devel, gsound-devel, vte291-devel, fprintd-devel, libzapojit-devel,
 bamf-devel, gstreamer1-vaapi-devel-docs, libfm-devel-docs, dbus-doc,
 gst-devtools-devel, harfbuzz-devel, libgdl-devel, rasqal-devel,
 template-glib-devel, parole-devel, xed-doc, libxmlb-devel, nautilus-
 python-devel, libpeas-devel, goocanvas2-devel, cutter-devel, gobject-
 introspection-devel, libxfce4util-devel, libgnome-keyring-devel,
 gnome-desktop3-devel, libdazzle-devel, unique-devel, libldm-devel,
 gtk3-devel-docs, yelp-devel, geoclue2-devel, libgtop2-devel, gcr-
 devel, libchamplain-devel, libgxps-devel, glade3-libgladeui-devel,
 gtranslator, gegl04-devel-docs, gtkimageview-devel, gimp-devel,
 cheese-libs-devel, gocl-devel, jsonrpc-glib-devel, libgweather-devel,
 buildstream-docs, libgit2-glib-devel, upower-devel-docs, intel-gpu-
 tools, dconf-devel, libgpod-doc, libu2f-host-devel, libsoup-devel,
 glade-devel, maliit-framework-docs, libgda-devel, p11-kit-devel,
 gtksourceview4-devel, lasem-devel, libmodulemd-devel, webkit2gtk3-doc,
 gnome-bluetooth-libs-devel, gom-devel, librsvg2-devel, libwnck3-devel,
 polkit-docs, libcryptui-devel, libcloudproviders-devel, libxklavier-
 devel, libhandy-devel, tracker-docs, gnome-todo-devel, gnome-calendar,
 anaconda-widgets-devel, json-glib-devel, geocode-glib-devel, gnome-
 desktop-devel, tumbler-devel, xreader-doc, libwnck-devel, udisks-
 devel, gmime30-devel, gtk2-devel-docs, gstreamer-plugins-base-devel-
 docs, poppler-glib-doc, ORBit2-devel, eog-devel, redland-devel,
 gstreamer1-rtsp-server-devel-docs, tomoe-devel, libidn2-devel,
 libsecret-devel, amtk-devel, latexila-doc, tepl-devel, libgdata-
 devel), /usr/share/gtk-doc/html(vips-devel, iio-sensor-proxy-docs,
 garcon-devel, libxml2-devel, libcanberra-devel, gedit-devel, gtef-
 devel, libunity-gtk-parser-devel, gtk4-devel-docs, gtkspell3-devel,
 exo-devel, libgnome-devel, u

[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #7 from Pete Walter  ---
Thanks, added appdata validation and the missing dconf-devel BuildRequires.

Spec URL: https://pwalter.fedorapeople.org/gnome-latex.spec
SRPM URL: https://pwalter.fedorapeople.org/gnome-latex-3.32.0-2.fc31.src.rpm

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #6 from Phil Wyett  ---
(In reply to Robert-André Mauchin from comment #5)
>  - Using the tepl 4.2.0 from the PR, I still get an error from OP's SPEC:
> 
> 
> Checking for DCONF_DEP... no
> BUILDSTDERR: configure: error: Package requirements (dconf) were not met:
> BUILDSTDERR: Package 'dconf', required by 'virtual:world', not found
> BUILDSTDERR: Consider adjusting the PKG_CONFIG_PATH environment variable if
> you
> BUILDSTDERR: installed software in a non-standard prefix.
> BUILDSTDERR: Alternatively, you may set the environment variables
> DCONF_DEP_CFLAGS
> BUILDSTDERR: and DCONF_DEP_LIBS to avoid the need to call pkg-config.
> BUILDSTDERR: See the pkg-config man page for more details.
> BUILDSTDERR: error: Bad exit status from /var/tmp/rpm-tmp.uINJ3a (%build)
> BUILDSTDERR: Bad exit status from /var/tmp/rpm-tmp.uINJ3a (%build)

If you use my original or latest spec file. You will not get the error.

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #5 from Robert-André Mauchin  ---
 - Using the tepl 4.2.0 from the PR, I still get an error from OP's SPEC:


Checking for DCONF_DEP... no
BUILDSTDERR: configure: error: Package requirements (dconf) were not met:
BUILDSTDERR: Package 'dconf', required by 'virtual:world', not found
BUILDSTDERR: Consider adjusting the PKG_CONFIG_PATH environment variable if you
BUILDSTDERR: installed software in a non-standard prefix.
BUILDSTDERR: Alternatively, you may set the environment variables
DCONF_DEP_CFLAGS
BUILDSTDERR: and DCONF_DEP_LIBS to avoid the need to call pkg-config.
BUILDSTDERR: See the pkg-config man page for more details.
BUILDSTDERR: error: Bad exit status from /var/tmp/rpm-tmp.uINJ3a (%build)
BUILDSTDERR: Bad exit status from /var/tmp/rpm-tmp.uINJ3a (%build)

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #4 from Phil Wyett  ---
I have updated the spec file to 3.32.0-2 on:

http://rpm.kathenas.org/fedora/testing/

This incorporates changes by submitter and subsequent suggestions.

Feel free to discard %changelog. :-D

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309



--- Comment #3 from Phil Wyett  ---
I have a WIP for this very update. Maybe we can combine work.

http://rpm.kathenas.org/fedora/testing/

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309

Phil Wyett  changed:

   What|Removed |Added

 CC||philwy...@kathenas.org



--- Comment #2 from Phil Wyett  ---
There is a PR for updating TEPL to 4.2.

https://src.fedoraproject.org/rpms/tepl/pull-request/1

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1688309] Review Request: gnome-latex - Integrated LaTeX Environment for the GNOME desktop

2019-03-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1688309

Robert-André Mauchin  changed:

   What|Removed |Added

 CC||zebo...@gmail.com



--- Comment #1 from Robert-André Mauchin  ---

 - {_datadir}/metainfo → %{_metainfodir}

 - You also need to validate the Appdata:

BuildRequires: libappstream-glib

[…]

appstream-util validate-relax --nonet
%{buildroot}%{_datadir}/metainfo/org.gnome.gnome-latex.appdata.xml

 - Build fails:

checking for DEP... no
BUILDSTDERR: configure: error: Package requirements (
BUILDSTDERR:glib-2.0 >= 2.56
BUILDSTDERR:gio-2.0 >= 2.56
BUILDSTDERR:gtk+-3.0 >= 3.22
BUILDSTDERR:gtksourceview-4 >= 4.0
BUILDSTDERR:tepl-4 >= 4.2
BUILDSTDERR:gspell-1 >= 1.8
BUILDSTDERR:gee-0.8 >= 0.10
BUILDSTDERR:gsettings-desktop-schemas
BUILDSTDERR: ) were not met:
BUILDSTDERR: Package 'tepl-4', required by 'virtual:world', not found
BUILDSTDERR: Consider adjusting the PKG_CONFIG_PATH environment variable if you
BUILDSTDERR: installed software in a non-standard prefix.
BUILDSTDERR: Alternatively, you may set the environment variables DEP_CFLAGS
BUILDSTDERR: and DEP_LIBS to avoid the need to call pkg-config.
BUILDSTDERR: See the pkg-config man page for more details.
BUILDSTDERR: error: Bad exit status from /var/tmp/rpm-tmp.c3Rz6s (%build)
BUILDSTDERR: Bad exit status from /var/tmp/rpm-tmp.c3Rz6s (%build)


tepl is only at version 3.0.0 in Rawhide. You need to contact the maintainer of
propose a PR to update to 4.2.0

-- 
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org