[Bug 576685] Review Request: pekwm - A small and flexible window manager

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=576685

--- Comment #13 from Germán Racca <gracca@xxxxxxxxx> 2010-05-11 02:02:12 EDT ---
(In reply to comment #11)

> This is what I would have done: At the end of %install, I would add
> 
> # Install docs (needed for contrib folder)
> mkdir -p %{buildroot}%{_docdir}/%{name}-%{version}/contrib
> install -p AUTHORS ChangeLog ChangeLog.aewm++ ChangeLog.until-0.1.6 LICENSE
> NEWS README \
>   %{buildroot}%{_docdir}/%{name}-%{version}
> install -p contrib/lobo/README contrib/lobo/check.png
> contrib/lobo/pekwm_autoprop.pl \
>   contrib/lobo/pekwm_menu_config.pl contrib/lobo/pekwm_menu_config.pl.vars \
>   contrib/lobo/uncheck.png %{buildroot}%{_docdir}/%{name}-%{version}/contrib
> 
> and in %files
> 
> %doc %{_docdir}/%{name}-%{version}
> 
> But your way is much simpler and works fine too. Nice idea actually. ;)

I also thought something similar before doing it this way.

> Congratulations, the package is APPROVED. Good work!

Thank you very much!

-- 
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@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]