Bug 620181
Summary: | Review Request: pastebinit - command line tool to pastebin file | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Lameire Alexis <alexis.lameire> |
Component: | Package Review | Assignee: | Mamoru TASAKA <mtasaka> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | fedora-package-review, mtasaka, notting, pikachu.2014 |
Target Milestone: | --- | Flags: | mtasaka:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | pastebinit-1.1-4.fc14 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-08-25 18:31:08 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Lameire Alexis
2010-08-01 12:57:59 UTC
Before checking this package: (In reply to comment #0) > All warnings are false positives, especially the spelling issues; the /etc/* > files should not be edited, so in this case the %config macro is not needed. - Then these files currently under /etc should be moved to /usr/share (%_datadir). Files under /etc must be used for actually "configuration" files, and if the files are not expected to be modified, /usr/share should be used instead of /etc. http://www.pathname.com/fhs/pub/fhs-2.3.html#ETCHOSTSPECIFICSYSTEMCONFIGURATION http://www.pathname.com/fhs/pub/fhs-2.3.html#USRSHAREARCHITECTUREINDEPENDENTDATA - Also I fail to understand why this package uses "pastebin.d" name for directory name. I guess %{_datadir}/%{name} (/usr/share/pastebinit) is more suitable to avoid potential namespace conflict. This dir is hardcoded on the script. I can't do anything unless I patch the sources. That is what I will do and by the way I will warn the developers about this issue Thanks for having a look at my package. New version with your suggestions; I submitted your fixes upstream: https://bugs.launchpad.net/pastebinit/+bug/62192 SPEC : http://pikachu.2014.free.fr/public/alexises/packaging/pastebinit/1.1-2/pastebinit.spec SRPM : http://pikachu.2014.free.fr/public/alexises/packaging/pastebinit/1.1-2/pastebinit-1.1-2.fc13.src.rpm rpmlint output : [packager@gnu SRPMS]$ rpmlint pastebinit-1.1-2.fc13.src.rpm pastebinit.src: W: spelling-error Summary(en_US) pastebin -> paste bin, paste-bin, pasteboard pastebinit.src: W: spelling-error %description -l en_US pastebin -> paste bin, paste-bin, pasteboard 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [packager@gnu SRPMS]$ rpmlint ../RPMS/noarch/pastebinit-1.1-2.fc13.noarch.rpm pastebinit.noarch: W: spelling-error Summary(en_US) pastebin -> paste bin, paste-bin, pasteboard pastebinit.noarch: W: spelling-error %description -l en_US pastebin -> paste bin, paste-bin, pasteboard 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [packager@gnu SRPMS]$ All are done :) ready to recheck, more speed them pizza hut :D Some comments: * BuildRoot - For Fedora and EPEL6, BuildRoot tag is no longer needed: https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag * "sed"ing ---------------------------------------------------------------------- sed "s|/etc/pastebin.d|%{_datadir}%{name}|g" %{name} ---------------------------------------------------------------------- - This line does nothing (just outputs the result of sed to stdout) Perhaps you meant "sed -i" - %{_datadir}%{name} is expanded as /usr/sharepastebinit (%{_datadir} = /usr/share, not /usr/share/) - Also perhaps you want to modify README file, and "/.pastebin.d" in pastebinit and README. Hi, thanks again for your comment. Here is the new version: SPEC : http://pikachu.2014.free.fr/public/alexises/packaging/pastebinit/1.1-3/pastebinit.spec SRPM : http://pikachu.2014.free.fr/public/alexises/packaging/pastebinit/1.1-3/pastebinit-1.1-3.fc13.src.rpm - solved the -i and the missing / - added new sed to modify the name of the user config. file (~/.pastebinit.d -> ~/.pastebinit) - moved an example conf. file to %{_docdir} - deleted BuildRootTag. I hope this should be OK now :) Almost okay. For -3: * Macros ---------------------------------------------------------------- xsltproc -''-nonet /usr/share/sgml/docbook/xsl-stylesheets*/manpages/docbook.xsl pastebinit.xml ---------------------------------------------------------------- - Please use %{_datadir} for /usr/share here. ---------------------------------------------------------- This package (pastebinit) is APPROVED by mtasaka ---------------------------------------------------------- Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from "Get a Fedora Account". After you create your account on FAS (Fedora Account System), please write your account name on this bug. Then I will sponsor you. If you want to import this package into Fedora 12/13/14, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on koji Fedora rebuilding system). When using Fedora SCM system, please check below for reference: http://fedoraproject.org/wiki/Using_Fedora_GIT If you have questions, please ask me. I have already a Fas account, my username is alexises. thenks for approuving my job. Okay, now I am sponsoring you. Please follow "Join" wiki again. New Package SCM Request ======================= Package Name: pastebinit Short Description: command line tool to pastebin file Owners: alexises Branches: f13 f14 InitialCC: Git done (by process-git-requests). pastebinit-1.1-4.fc14 has been submitted as an update for Fedora 14. http://admin.fedoraproject.org/updates/pastebinit-1.1-4.fc14 pastebinit-1.1-4.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/pastebinit-1.1-4.fc13 Closing. pastebinit-1.1-4.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. pastebinit-1.1-4.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. |