Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

Allow for summary to be different from description #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

djnym
Copy link
Contributor

@djnym djnym commented May 8, 2017

Simple change should be pretty self explanatory.

@djnym
Copy link
Contributor Author

djnym commented May 24, 2017

Hopefully this patch is the least controversial, it simply adds a new switch to provide a different summary from description. But again testing under the same suse installation as the previous ones. Without the patch

> ../epm.erl -f -t rpm -n test -v 1.1.0 -a x86_64 --description "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum." tmp etc usr
> rpm -qpi test-1.1.0-1.x86_64.rpm
Name        : test
Version     : 1.1.0
Release     : 1
Architecture: x86_64
Install Date: (not installed)
Group       : (none)
Size        : 212

Signature   : (none)
Source RPM  : (none)
Build Date  : Wed 24 May 2017 03:16:57 PM PDT
Build Host  : localhost
Relocations : (not relocatable)
Summary     : Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
Description :
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
Distribution: (none)

With the patch (but without the option)

> ../epm.erl -f -t rpm -n test -v 1.1.0 -a x86_64 --description "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum." tmp etc usr
> > rpm -qpi test-1.1.0-1.x86_64.rpm
Name        : test
Version     : 1.1.0
Release     : 1
Architecture: x86_64
Install Date: (not installed)
Group       : (none)
Size        : 212

Signature   : (none)
Source RPM  : (none)
Build Date  : Wed 24 May 2017 03:18:27 PM PDT
Build Host  : localhost
Relocations : (not relocatable)
Summary     : no summary
Description :
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
Distribution: (none)

and with the patch and the summary option added

> ../epm.erl -f -t rpm -n test -v 1.1.0 -a x86_64 --summary "Lorem ipsum dolor" --description "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum." tmp etc usr
>  rpm -qpi test-1.1.0-1.x86_64.rpm
Name        : test
Version     : 1.1.0
Release     : 1
Architecture: x86_64
Install Date: (not installed)
Group       : (none)
Size        : 212

Signature   : (none)
Source RPM  : (none)
Build Date  : Wed 24 May 2017 03:19:21 PM PDT
Build Host  : localhost
Relocations : (not relocatable)
Summary     : Lorem ipsum dolor
Description :
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
Distribution: (none)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant