[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
   Fixed In Version||corebird-0.5-2.fc20
 Resolution|--- |ERRATA
Last Closed||2013-12-13 22:09:59



--- Comment #25 from Fedora Update System upda...@fedoraproject.org ---
corebird-0.5-2.fc20 has been pushed to the Fedora 20 stable repository.

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #15 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
[+] OK
[-] NA
[?] Issue

** Mandatory review guidelines: **
 [+] rpmlint output:
[asinha@ankur-laptop  SRPMS]$ rpmlint
/var/lib/mock/fedora-rawhide-x86_64/result/*.rpm ../SPECS/corebird.spec
./corebird-0.5-1.fc20.src.rpm
corebird.src: W: invalid-url Source0:
https://github.com/baedert/corebird/archive/corebird-0.5.tar.gz HTTP Error 404:
Not Found
corebird.x86_64: W: no-manual-page-for-binary corebird 
../SPECS/corebird.spec: W: invalid-url Source0:
https://github.com/baedert/corebird/archive/corebird-0.5.tar.gz HTTP Error 404:
Not Found
corebird.src: W: invalid-url Source0:
https://github.com/baedert/corebird/archive/corebird-0.5.tar.gz HTTP Error 404:
Not Found
4 packages and 1 specfiles checked; 0 errors, 4 warnings.
[asinha@ankur-laptop  SRPMS]$
 [+] License is acceptable (...)
 [+] License field in spec is correct
 [+] License files included in package %docs if included in source package
 [+] License files installed when any subpackage combination is installed
 [+] Spec written in American English
 [+] Spec is legible
 [+] Sources match upstream unless altered to fix permissibility issues
 [+] Build succeeds on at least one primary arch
 [+] Build succeeds on all primary arches or has ExcludeArch + bugs filed
 [+] BuildRequires correct, justified where necessary
 [-] Locales handled with %find_lang, not %_datadir/locale/*
 [+] %post, %postun call ldconfig if package contains shared .so files
 [+] No bundled libs
 [-] Relocatability is justified
 [?] Package owns all directories it creates
^
Please also own the appdata directory. It's OK to co-own it since
gnome-software isn't required for functioning of this package.

 [+] Package requires others for directories it uses but does not own
 [+] No duplication in %files unless necessary for license files
 [+] File permissions are sane
 [+] Package contains permissible code or content
 [-] Large docs go in -doc subpackage
 [+] %doc files not required at runtime
 [-] Static libs go in -static package/virtual Provides
 [-] Development files go in -devel package
 [-] -devel packages Require base with fully-versioned dependency, %_isa
 [+] No .la files
 [+] GUI app uses .desktop file, installs it with desktop-file-install
 [+] File list does not conflict with other packages' without justification
 [+] File names are valid UTF-8

** Optional review guidelines: **
 [-] Query upstream about including license files
 [-] Translations of description, summary
 [+] Builds in mock
 [+] Builds on all arches
 [+] Functions as described (e.g. no crashes)
 [+] Scriptlets are sane
 [-] Subpackages require base with fully-versioned dependency if sensible
 [-] .pc file subpackage placement is sensible
 [+] No file deps outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin
 [-] Include man pages if available

Naming guidelines:
 [+] Package names use only a-zA-Z0-9-._+ subject to restrictions on -._+
 [+] Package names are sane
 [+] No naming conflicts
 [+] Spec file name matches base package name
 [+] Version is sane
 [+] Version does not contain ~
 [+] Release is sane
 [+] %dist tag
 [+] Case used only when necessary
 [+] Renaming handled correctly

Packaging guidelines:
 [+] Useful without external bits
 [+] No kmods
 [-] Pre-built binaries, libs removed in %prep
 [+] Sources contain only redistributable code or content
 [+] Spec format is sane
 [+] Package obeys FHS, except libexecdir, /run, /usr/target
 [+] No files in /bin, /sbin, /lib* on = F17
 [-] Programs run before FS mounting use /run instead of /var/run
 [-] Binaries in /bin, /sbin do not depend on files in /usr on  F17
 [+] No files under /srv, /opt, /usr/local
 [+] Changelog in prescribed format
 [+] No Packager, Vendor, Copyright, PreReq tags
 [+] Summary does not end in a period
 [-] Correct BuildRoot tag on  EL6
 [-] Correct %clean section on  EL6
 [+] Requires correct, justified where necessary
[asinha@ankur-laptop  result]$ review-req-check
== corebird-0.5-1.fc21.src.rpm ==
Provides:

Requires:
gtk3-devel = 3.9
glib2-devel = 2.38
rest-devel
json-glib-devel
libnotify-devel
sqlite-devel
libsoup-devel
vala-devel
cmake
librsvg2-tools
desktop-file-utils
libgee-devel

== corebird-0.5-1.fc21.x86_64.rpm ==
Provides:
application()
application(corebird.desktop)
corebird = 0.5-1.fc21
corebird(x86-64) = 0.5-1.fc21

Requires:
/bin/sh
/bin/sh
/bin/sh
hicolor-icon-theme
libatk-1.0.so.0()(64bit)
libc.so.6()(64bit)
libc.so.6(GLIBC_2.14)(64bit)
libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.4)(64bit)
libcairo-gobject.so.2()(64bit)
libcairo.so.2()(64bit)
libgdk-3.so.0()(64bit)
libgdk_pixbuf-2.0.so.0()(64bit)
libgee-0.8.so.2()(64bit)

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Ryan Lerch rle...@redhat.com changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)



--- Comment #16 from Ryan Lerch rle...@redhat.com ---
(In reply to Ankur Sinha (FranciscoD) from comment #15)
 
 PACKAGE IS *** APPROVED ***
 
 Tiny issues you should correct before importing:
 
 - No need to remove fonts from the ./data directory. They aren't included in
   the tar any more.

Okay, will remove the following lines from the spec:

# in separate package
rm -rf %{buildroot}%{_datadir}/fonts

 - Please co-own the appdata directory

Not 100% sure what to do to achieve this.


Also, I originally forgot to make this block the NEED_SPONSOR bug, as this is
my first package. 

cheers,
ryanlerch


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Ralph Bean rb...@redhat.com changed:

   What|Removed |Added

 CC||rb...@redhat.com



--- Comment #17 from Ralph Bean rb...@redhat.com ---
I am sponsoring ryanlerch into the packager group.

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Ralph Bean rb...@redhat.com changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #18 from Ryan Lerch rle...@redhat.com ---
I made the following changes to the spec to solve the last 2 final issues:


@@ -1,6 +1,6 @@
 Name:corebird
 Version:0.5
-Release:1%{?dist}
+Release:2%{?dist}
 Summary:Native GTK Twitter client
 License:GPLv3+
 URL:http://corebird.baedert.org/
@@ -36,9 +36,6 @@

 desktop-file-validate %{buildroot}/%{_datadir}/applications/%{name}.desktop

-# in separate package
-rm -rf %{buildroot}%{_datadir}/fonts
-
 %post
 /bin/touch --no-create %{_datadir}/icons/hicolor /dev/null || :

@@ -60,9 +57,13 @@
 %{_datadir}/%{name}/
 %{_datadir}/glib-2.0/schemas/org.baedert.%{name}.gschema.xml
 %{_datadir}/icons/hicolor/*/apps/%{name}.png
-%{_datadir}/appdata/%{name}.appdata.xml
+%{_datadir}/appdata/

 %changelog
+* Tue Dec 03 2013 Ryan Lerch ryanle...@fedoraproject.org 0.5-2
+- Removed line from the spec that removed the old font files
+- Changed the spec so the appdata dir is co-owned 
+
 * Fri Nov 29 2013 Ankur Sinha ankursinha AT fedoraproject DOT org 0.5-1
 - Update to 0.5



-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Ryan Lerch rle...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #19 from Ryan Lerch rle...@redhat.com ---
New Package SCM Request
===
Package Name: corebird
Short Description: Native GTK Twitter client
Owners: ryanlerch
Branches: f20
InitialCC: threebean

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |



-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #20 from Jon Ciesla limburg...@gmail.com ---
threebean is not a vaid FAS account.

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #21 from Ralph Bean rb...@redhat.com ---
Doh.  I should've caught that.  It should have been 'ralph'.

Ryan, can you re-do the scm admin request changing the initialcc from threebean
to ralph?

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Ryan Lerch rle...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #22 from Ryan Lerch rle...@redhat.com ---
New Package SCM Request
===
Package Name: corebird
Short Description: Native GTK Twitter client
Owners: ryanlerch
Branches: f20
InitialCC: ralph

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #23 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #24 from Fedora Update System upda...@fedoraproject.org ---
corebird-0.5-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/corebird-0.5-2.fc20

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-12-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #14 from Ryan Lerch rle...@redhat.com ---
Thanks ankur!

the updated files are available here:
http://ryanlerch.org/corebird/package/

cheers,
ryanlerch

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed:

   What|Removed |Added

 Depends On|1026376 |



--- Comment #13 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
Hi,

Upstream has released version 0.5 which does not depend on fontawesome and also
contains a bunch of fixes and feature additions. I think the app is now in good
enough state for end users, so we could continue the review and quickly get it
into the Fedora repositories. 

Ryan, here's an updated spec for 0.5
(http://ankursinha.fedorapeople.org/corebird.spec) that you can refer 

Could you please submit an updated srpm that I could now formally review?

Thanks,
Warm regards,
Ankur


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1026376
[Bug 1026376] Review Request: fontawesome-fonts - Iconic font set
-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Heiko Adams bugzi...@heiko-adams.de changed:

   What|Removed |Added

 CC||bugzi...@heiko-adams.de



--- Comment #12 from Heiko Adams bugzi...@heiko-adams.de ---
FYI: With todays commit #9fea73447ebb86d82242e104d7804d78e221dbe8 corebird no
longer uses fontawesome:
https://github.com/baedert/corebird/commit/9fea73447ebb86d82242e104d7804d78e221dbe8

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #9 from Ryan Lerch rle...@redhat.com ---
(In reply to Christopher Meng from comment #8)
 To me all scriptlets being used or mentioned in that page of wiki, I'd
 suggest that:
 
 Either from
 
 /bin/touch --no-create %{_datadir}/icons/hicolor /dev/null || :
 
 to
 
 touch --no-create %{_datadir}/icons/hicolor /dev/null || :
 
 Or from
 
 /bin/touch --no-create %{_datadir}/icons/hicolor /dev/null || :
 
 to
 
 %{_bindir}/touch --no-create %{_datadir}/icons/hicolor /dev/null || :

Hi Christopher!

specifically what wikipage are you referring to?

regards,
ryanlerch

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #10 from Christopher Meng cicku...@gmail.com ---
(In reply to Ryan Lerch from comment #9)
 Hi Christopher!
 
 specifically what wikipage are you referring to?
 
 regards,
 ryanlerch

I mean this:

https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #11 from Ryan Lerch rle...@redhat.com ---
(In reply to Christopher Meng from comment #10)
 (In reply to Ryan Lerch from comment #9)
  Hi Christopher!
  
  specifically what wikipage are you referring to?
  
  regards,
  ryanlerch
 
 I mean this:
 
 https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache

the example in that section of that wikipage suggests to use /bin/touch
rather than just touch. What is the rationale for suggesting this change?

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 CC||cicku...@gmail.com



--- Comment #8 from Christopher Meng cicku...@gmail.com ---
To me all scriptlets being used or mentioned in that page of wiki, I'd suggest
that:

Either from

/bin/touch --no-create %{_datadir}/icons/hicolor /dev/null || :

to

touch --no-create %{_datadir}/icons/hicolor /dev/null || :

Or from

/bin/touch --no-create %{_datadir}/icons/hicolor /dev/null || :

to

%{_bindir}/touch --no-create %{_datadir}/icons/hicolor /dev/null || :

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #7 from Ryan Lerch rle...@redhat.com ---
Thanks Ankur!

I have updated the SRPM and the spec with this patch.

http://ryanlerch.fedorapeople.org/packaging/corebird/2/

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #4 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
Oh! It'll also be really great if you can write up an appdata file and include
it in the package, and even better if you could also send it upstream.

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #5 from Ryan Lerch rle...@redhat.com ---
(In reply to Ankur Sinha (FranciscoD) from comment #2)
 A 0.4 release was made 4 days ago. It's better to use this. You won't need
 to carry the git commit tag if you use this. I'll begin reviewing this
 version but will have to go over the updated srpm before I can approve the
 package
 
 https://github.com/baedert/corebird/releases/tag/0.4
 
 Thanks,
 Warm regards,
 Ankur

Thanks! i noticed this this morning too!

here is the updated spec  SRPM to for the 0.4 release:
http://ryanlerch.fedorapeople.org/packaging/corebird/corebird.spec
http://ryanlerch.fedorapeople.org/packaging/corebird/corebird-0.4-0.fc20.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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #6 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
Created attachment 820077
  -- https://bugzilla.redhat.com/attachment.cgi?id=820077action=edit
Git diff patch.

I'll go ahead and review the newer version. One nitpick, certainly not a
blocker:

Upstream forgot to bump the about-dialog version before tagging and did it in
subsequent commits:

* 003bee4 - (2 days ago) about dialog: 0.3 - 0.4 — Timm Bäder
* 42c0373 - (2 days ago) pkgbuild: Change _realver to 0.4 — Timm Bäder
* 7b82914 - (2 days ago) readme: Remove ubuntu/fedora librsvg dependencies —
Timm Bäder (tag: 0.4)

Can you look to incorporate the two tiny changes (patch attached)? I think a
simple sed command in %prep will do it. Or, you can use a tarball from this
commit (this might be overkill).

Users might get confused when they see a difference in version between the
about dialog and rpm -q output.

Detailed review coming up shortly. 

Thanks,
Warm regards,
Ankur

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

 Depends On||1026376




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1026376
[Bug 1026376] Review Request: fontawesome-fonts - Iconic font set
-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380

Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||sanjay.an...@gmail.com
   Assignee|nob...@fedoraproject.org|sanjay.an...@gmail.com
  Flags||fedora-review?



--- Comment #1 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
Hi,

I'll review corebird and the required font sets.

Thanks,
Ankur

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #2 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
A 0.4 release was made 4 days ago. It's better to use this. You won't need to
carry the git commit tag if you use this. I'll begin reviewing this version but
will have to go over the updated srpm before I can approve the package

https://github.com/baedert/corebird/releases/tag/0.4

Thanks,
Warm regards,
Ankur

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1026380] Review Request: corebird - Native GTK Twitter client

2013-11-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1026380



--- Comment #3 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com ---
(In reply to Ankur Sinha (FranciscoD) from comment #2)
 A 0.4 release was made 4 days ago.

*2 days ago. 

 It's better to use this. You won't need
 to carry the git commit tag if you use this. I'll begin reviewing this
 version but will have to go over the updated srpm before I can approve the
 package
 
 https://github.com/baedert/corebird/releases/tag/0.4
 
 Thanks,
 Warm regards,
 Ankur

I'd better get some coffee.

-- 
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://admin.fedoraproject.org/mailman/listinfo/package-review