Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: emacs-identica-mode - Identica mode for emacs.
https://bugzilla.redhat.com/show_bug.cgi?id=532241
Summary: Review Request: emacs-identica-mode - Identica mode for emacs. Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: sagarun@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://sagarun.fedorapeople.org/SPECS/emacs-identica-mode.spec SRPM URL: http://sagarun.fedorapeople.org/SRPMS/emacs-identica-mode-0.8-1.fc11.src.rpm Description: This package provides identica mode for emacs. Identi.ca is a free software microblogging service, similar to Twitter, based on the Laconi.ca code base.
Links to koji builds:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1780394 http://koji.fedoraproject.org/koji/taskinfo?taskID=1780396
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=532241
Arun SAG sagarun@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |shakthimaan@gmail.com Blocks| |177841(FE-NEEDSPONSOR)
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=532241
--- Comment #1 from Arun SAG sagarun@gmail.com 2009-10-31 15:07:52 EDT --- Testing the package:
Add the following lines to ~/.emacs file
;;; Identi.ca mode for emacs (require 'identica-mode) (setq identica-username "username") (setq identica-password "password") ;; Add this to send status updates (global-set-key "\C-cip" 'identica-update-status-interactive) ;; Add this to send direct messages (global-set-key "\C-cid" 'identica-direct-message-interactive)
How to: http://blog.nethazard.net/identica-mode-for-emacs/
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=532241
--- Comment #2 from Arun SAG sagarun@gmail.com 2009-12-14 03:36:22 EDT --- obsolete entries in spec file removed
Spec URL: http://sagarun.fedorapeople.org/SPECS/emacs-identica-mode.spec
SRPM URL: http://sagarun.fedorapeople.org/SRPMS/emacs-identica-mode-0.8-2.fc11.src.rpm
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=532241
Arun SAG sagarun@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|177841(FE-NEEDSPONSOR) |
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=532241
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status Whiteboard| |NotReady
--- Comment #3 from Jason Tibbitts tibbs@math.uh.edu 2010-11-03 14:01:16 EDT --- The SRPM link is invalid, though the spec still works. Unfortunately the spec is insufficient to recreate the package due to Source1, so this cannot be reviewed as is.
Please clear the Whiteboard if you provide working links, thanks.
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=532241
--- Comment #4 from Arun SAG sagarun@gmail.com 2010-11-03 14:12:43 EDT --- (In reply to comment #3)
The SRPM link is invalid, though the spec still works. Unfortunately the spec is insufficient to recreate the package due to Source1, so this cannot be reviewed as is.
Please clear the Whiteboard if you provide working links, thanks.
SRPM URL: http://sagarun.fedorapeople.org/SRPMS/emacs-identica-mode-0.8-2.fc12.src.rpm
Spec URL: http://sagarun.fedorapeople.org/SPECS/emacs-identica-mode.spec
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=532241
Michel Alexandre Salim michel+fdr@michelsylvain.info changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |michel+fdr@michelsylvain.in | |fo AssignedTo|nobody@fedoraproject.org |michel+fdr@michelsylvain.in | |fo Flag| |fedora-review?
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=532241
--- Comment #5 from Arun SAG sagarun@gmail.com 2010-11-16 21:03:10 EST --- Updated to the latest version: (the info file needs fix)
SRPM URL: http://sagarun.fedorapeople.org/SRPMS/emacs-identica-mode-1.0-1.fc13.src.rpm
Spec URL: http://sagarun.fedorapeople.org/SPECS/emacs-identica-mode.spec
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=532241
Michel Alexandre Salim michel+fdr@sylvestre.me changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(sagarun@gmail.com | |)
--- Comment #6 from Michel Alexandre Salim michel+fdr@sylvestre.me 2011-04-06 08:10:29 EDT --- Hi Arun,
Sorry for the late reply! A couple of problems with the package right now: - incomplete Build Requirement: you're using makeinfo, which is supplied by the texinfo package - Upstream releases unversioned tarballs on their main download site, which makes it hard for someone to verify that the sources are accurate. Suggest contacting upstream, and meanwhile downloading the tarballs generated from the Git tags instead (see detailed comments below)
Also, 1.1 is out now -- probably a good time to upgrade.
* TODO Review [72%] - [X] Names [2/2] - [X] Package name - [X] Spec name - [X] Package version [2/2] http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Package_Versioning - [X] Version number http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Version_Tag - [X] Release tag http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Release_Tag
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Pre-Release_package... - [X] Meets [[http://fedoraproject.org/wiki/Packaging/Guidelines%5D%5Bguidelines]] - [-] Source files match upstream Upstream does not put the versioned source tarball in their download server; I suggest downloading the tarballs generated from the Git tags instead: http://git.savannah.gnu.org/cgit/identica-mode.git
Contacting upstream is probably a good idea too, asking for the download tarballs to carry the version number - [X] [[http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries%5D%5BNo bundled libraries]] - [X] License [4/4] - [X] License is Fedora-approved - [X] No licensing conflict - [X] License field accurate - [X] License included iff packaged by upstream - [-] rpmlint [1/2] - [X] on src.rpm emacs-identica-mode.src: W: file-size-mismatch identica-mode.tar.gz = 31463, http://download.savannah.gnu.org/releases/identica-mode/identica-mode.tar.gz = 33026 1 packages and 0 specfiles checked; 0 errors, 1 warnings.
This is due to the source file problem, see above. Otherwise fine - [ ] on x86_64.rpm - [X] Language & locale [2/2] - [X] Spec in US English - [X] Spec legible - [-] Build [2/3] - [-] Koji results http://koji.fedoraproject.org/koji/taskinfo?taskID=2978357 - [-] BRs complete makeinfo not found; needs to BR on texinfo - [X] Directory ownership - [X] Spec inspection [9/9] - [X] No duplicate files - [X] File permissions - [X] Filenames must be UTF-8 - [X] no BuildRoot ([[https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag%5D%5Bexcep... if targeting RHEL5]]) - [X] [RHEL] %clean section https://fedoraproject.org/wiki/Packaging/Guidelines#.25clean) - [X] [RHEL 5] %buildroot cleaned on %install - [X] Macro usage consistent - [X] Documentation [1/1] - [X] %doc files are non-essential - [X] Other subpackages - [X] [[http://fedoraproject.org/wiki/Packaging/ScriptletSnippets%5D%5BScriptlets]] [1/1] - [X] info files
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=532241
Arun SAG sagarun@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(sagarun@gmail.com | |) |
--- Comment #7 from Arun SAG sagarun@gmail.com 2011-06-12 04:33:26 EDT --- Hi Michel,
Sorry for the late reply. Updated to git tag v 1.1
SRPM URL: http://sagarun.fedorapeople.org/SRPMS/emacs-identica-mode-1.1-1.fc15.src.rpm
SPEC URL: http://sagarun.fedorapeople.org/SPECS/emacs-identica-mode.spec
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=532241
--- Comment #8 from Arun SAG sagarun@gmail.com 2011-08-18 03:42:31 EDT --- Ping
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=532241
Arun SAG sagarun@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |WONTFIX Last Closed| |2012-01-21 15:15:10
--- Comment #9 from Arun SAG sagarun@gmail.com 2012-01-21 15:15:10 EST --- Closing this. No one seems to be interested in reviewing this!
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=532241
Arun SAG sagarun@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review-
package-review@lists.fedoraproject.org