[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2012-08-26 Thread bugzilla 
https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #13 from Mario Blättermann  ---
The latest commits to Buzzbird's Git repo [1] are older than two years, that's
why we can safely assume that upstream is dead. I don't expect to get a working
package in Fedora ever, because we can't screw down the gecko-libs version.
This review request should either be closed as FE-DEADREVIEW or we need an
additional gecko-libs < 1.9.3 which installs parallel to the current one.

[1] https://github.com/mdesjardins/buzzbird

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2012-08-27 Thread bugzilla 
https://bugzilla.redhat.com/show_bug.cgi?id=668204

Jan Klepek  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Blocks||201449 (FE-DEADREVIEW)
 Resolution|--- |WONTFIX
 Whiteboard|NotReady|
Last Closed||2012-08-27 04:29:49

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-05-08 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

Mario Blättermann  changed:

   What|Removed |Added

 CC||mari...@freenet.de

--- Comment #9 from Mario Blättermann  2011-05-08 12:28:21 
EDT ---
(In reply to comment #7)
> this does not apply to source tarball as far as I know, no 
> bundled/pre-compiled
> binary file is present in final rpm.

But you have to make sure that no one of the prebuilt binaries will be used
during the build process. Otherwise, the resulting binaries would be affected
by these libraries anyway. The solution is to remove the binaries before you do
anything in your package. If it doesn't build anymore, we have a problem. If it
builds properly again, then we know that all is OK.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-05-09 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #10 from Jan Klepek  2011-05-09 03:26:11 EDT 
---
(In reply to comment #9)
> (In reply to comment #7)
> > this does not apply to source tarball as far as I know, no 
> > bundled/pre-compiled
> > binary file is present in final rpm.
> 
> But you have to make sure that no one of the prebuilt binaries will be used
> during the build process. Otherwise, the resulting binaries would be affected
> by these libraries anyway. The solution is to remove the binaries before you 
> do
> anything in your package. If it doesn't build anymore, we have a problem. If 
> it
> builds properly again, then we know that all is OK.

Which is not case of this review request...due to fact that %build is empty.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-05-09 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #11 from Christopher Aillon  2011-05-09 
16:50:20 EDT ---
Sure, but you're generating the source tarball anyway, stripping out the binary
cruft isn't that much more difficult and would probably make reviewers happier
(at least it would make me happier).

Anyway, a note that due to the gecko requires, this will not work on F15 or
newer.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-01-08 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #1 from Jan Klepek  2011-01-08 18:55:16 EST 
---
rpmlint output: 
buzzbird.noarch: W: spelling-error Summary(en_US) identi -> identic, identify,
identity
buzzbird.noarch: W: spelling-error %description -l en_US identi -> identic,
identify, identity
buzzbird.noarch: W: spelling-error %description -l en_US gd -> dg, Gd, hd
buzzbird.src: W: spelling-error Summary(en_US) identi -> identic, identify,
identity
buzzbird.src: W: spelling-error %description -l en_US identi -> identic,
identify, identity
buzzbird.src: W: spelling-error %description -l en_US gd -> dg, Gd, hd
buzzbird.src: W: invalid-url Source0: buzzbird-0.9.1.tar.gz

spelling errors are false positive, and upstream does not provide any URL to
source packages, they are fetched from git.

scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2709978

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-02-16 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #2 from Christopher Aillon  2011-02-16 15:43:17 
EST ---
* Your source tarball includes a few binary files including a windows EXE file
that need to get stripped.  Also, if you're creating the source tarball
yourself, please consider using bzip2 or xz, which will offer better
compression (smaller tarballs).

* I looked at application.ini which claims

[Gecko]
MinVersion=1.8
MaxVersion=1.9.2.*

So please have your spec
  Requires: gecko-libs >= 1.8
  Requires: gecko-libs <  1.9.3

* If you're creating a .desktop file from scratch, just save it in its own
file, and include it as a Source2, like you're doing with the man page.  And
send both to upstream so they can include them in the future.

* Is the extensions a Firefox extension?  Based on extensions/install.rdf it
appears to be since it uses Firefox's Application ID.  If so, it needs to be
installed into
%{_libdir}/mozilla/extensions/{ec8030f7-c20a-464f-9b0e-13a3a9e97384}/ and
perhaps subpackaged?  But unless I'm missing something, it's just a noop now,
and can probably get dropped?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-03-08 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #3 from Jan Klepek  2011-03-08 03:34:36 EST 
---
Hi,

1] buzzbird.exe is not present in final rpm, so what's your point?
2] requires gecko - done
3] desktop file: for me it is easy to maintain it in spec file, changed to
source2
4] firefox extension: i will check, not sure about it

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-03-08 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #4 from Christopher Aillon  2011-03-08 09:58:56 
EST ---
(In reply to comment #3)
> 1] buzzbird.exe is not present in final rpm, so what's your point?

We ship the SRPM too.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-03-08 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #5 from Jan Klepek  2011-03-08 10:49:03 EST 
---
(In reply to comment #4)
> (In reply to comment #3)
> > 1] buzzbird.exe is not present in final rpm, so what's your point?
> 
> We ship the SRPM too.

I'm aware of that, but existence of .exe file does not affect anybody anyhow.
It is just sitting there and doing nothing. Just like .bat file there and other
files which are not relevant for fedora. If I would like to exclude all of
them, it will lead to script which has to be checked/modified every release.
Packaging guidelines does not say that source tarball has to contain only files
relevant to Fedora. If you are concerned about size, I could switch to newly
created tarballs from upstream which are even bigger (as they have xulrunner
inside).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-03-08 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #6 from Christopher Aillon  2011-03-08 13:35:41 
EST ---
http://fedoraproject.org/wiki/Packaging/Guidelines#No_inclusion_of_pre-built_binaries_or_libraries

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-03-09 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #7 from Jan Klepek  2011-03-09 15:16:46 EST 
---
this does not apply to source tarball as far as I know, no bundled/pre-compiled
binary file is present in final rpm.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-03-10 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

--- Comment #8 from Jan Klepek  2011-03-10 06:29:01 EST 
---
updated spec/srpm
Spec URL: http://hpejakle.fedorapeople.org/packages/buzzbird.spec
SRPM URL:
http://hpejakle.fedorapeople.org/packages/buzzbird-0.9.1-2.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2012-04-25 Thread bugzilla 
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=668204

Jason Tibbitts  changed:

   What|Removed |Added

 CC|fedora-package-review@redha |package-review@lists.fedora
   |t.com   |project.org
  Status Whiteboard||NotReady

--- Comment #12 from Jason Tibbitts  2012-04-25 19:52:29 EDT 
---
While this still builds, it will not install on any Fedora version I tried due
to the depenency on gecko-libs < 1.9.3.  Marking as not ready for review;
please clear the whiteboard if providing a package which will work on a current
Fedora version.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review