spec: http://terjeros.fedorapeople.org/gdisk/gdisk.spec srpm: http://terjeros.fedorapeople.org/gdisk/gdisk-0.6.1-1.fc12.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1943397 desc: An fdisk-like partitioning tool for GPT disks. GPT fdisk features a command-line interface, fairly direct manipulation of partition table structures, recovery tools to help you deal with corrupt partition tables, and the ability to convert MBR disks to GPT format.
You can also use the Rod Smith's own src-rpm file from http://sourceforge.net/projects/gptfdisk/files/ . I propose this software be added to Fedora's Live and Install CDs and DVDs as well, since GPT is becoming commonplace and there is no other gpt tool that matches the features of GPT fdisk. I also request the Fedora Live CD team to include GParted apart from Palimpest Disk Utility.
I used Rod's spec as starting point. Updated to 0.6.2: spec: http://terjeros.fedorapeople.org/gdisk/gdisk.spec srpm: http://terjeros.fedorapeople.org/gdisk/gdisk-0.6.1-1.fc12.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1955015
Please add this software to Fedora's official rpm repositories as well to Fedora Install discs.
- 0.6.3 spec: http://terjeros.fedorapeople.org/gdisk/gdisk.spec srpm: http://terjeros.fedorapeople.org/gdisk/gdisk-0.6.3-1.fc12.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1985435
Fedora review gdisk-0.6.3-1.fc12.src.rpm 2010-03-11 + OK ! needs attention rpmlint output: gdisk.i686: W: spelling-error Summary(en_US) fdisk -> disk, frisk, f disk gdisk.i686: W: spelling-error %description -l en_US fdisk -> disk, frisk, f disk gdisk.src: W: spelling-error Summary(en_US) fdisk -> disk, frisk, f disk gdisk.src: W: spelling-error %description -l en_US fdisk -> disk, frisk, f disk 3 packages and 0 specfiles checked; 0 errors, 4 warnings. + rpmlint warnings above are harmless and can be ignored + The package is named according to the Package Naming Guidelines. + Spec file name matches the base package name + The package follows the Packaging Guidelines + The package is licensed with a Fedora approved license and meets the Licensing Guidelines. + The license field in the spec file matches the actual license + The package contains the license file (COPYING) + Spec file is written in American English + Spec file is legible + Upstream sources match sources in the srpm. md5sum: 0a43faf9c990fdf45d371dc623830e11 gdisk-0.6.3.tgz 0a43faf9c990fdf45d371dc623830e11 Download/gdisk-0.6.3.tgz + The package builds in koji n/a ExcludeArch bugs filed + BuildRequires look sane n/a The spec file MUST handle locales properly + Packages does not bundle copies of system libraries + Does not use Prefix: /usr + Package owns all directories it creates + No duplicate files in %files + Permissions are properly set and %files has %defattr + %clean contains %{__rm} -rf %{buildroot} + Consistent use of macros + Package contains code or permissible content n/a Large documentation files should go in -doc subpackage + Files marked %doc should not affect package n/a Header files should be in -devel n/a Static libraries should be in -static n/a Library files that end in .so must go in a -devel package n/a -devel must require the fully versioned base n/a Packages should not contain libtool .la files n/a Packages containing GUI apps must include %{name}.desktop file + Packages must not own files or directories owned by other packages + Filenames must be valid UTF-8 ! As per https://fedoraproject.org/wiki/Packaging/SourceURL#Sourceforge.net , use the following for Source0: Source0: http://downloads.sourceforge.net/gptfdisk/gdisk-%{version}.tgz I would suggest to not use %{__command} macros in new spec files, as they are really not needed and only make the spec file harder to read. Instead of %{__make}, %{__rm}, and %{__install} I'd use just plain make, rm, and install. But this is mostly just personal preference, and up to you if you want to change that. Package looks good and you can fix Source0 when importing into CVS. APPROVED
Will change the source url. Thanks for the review! New Package CVS Request ======================= Package Name: gdisk Short Description: An fdisk-like partitioning tool for GPT disks Owners: terjeros Branches: F-11 F-12 F-13 InitialCC:
Hi again Kalev, if bored, please consider #560728 :-)
CVS done (by process-cvs-requests.py).
Imported, built and pushed.
Would you mind maintaining this in EPEL 5, too? Afaics ar GPT partitions supported in RHEL, too: # grep EFI_PART /boot/config-2.6.18-164.* /boot/config-2.6.18-164.11.1.el5:CONFIG_EFI_PARTITION=y
Hi Till, at the moment I don't do epel builds, might change in the future, but not short term, sorry.
Package Change Request ====================== Package Name: gdisk New Branches: el5 el6 Owners: orion InitialCC: Since Terje doesn't want to maintain in epel, I will.
Git done (by process-git-requests).