[Bug 606759] Review Request: nvi - 4.4BSD re-implementation of vi

[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=606759

Michel Alexandre Salim <michael.silvanus@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |michael.silvanus@xxxxxxxxx
               Flag|                            |fedora-review?,
                   |                            |needinfo?(mcepl@xxxxxxxxxx)

--- Comment #1 from Michel Alexandre Salim <michael.silvanus@xxxxxxxxx> 2010-06-23 14:26:59 EDT ---
Reviewing this. Question: which nvi.spec is correct? The one you linked to is
newer than the one in the SRPM, and there are some differences:

$ diff -u nvi.spec ~/rpmbuild/SPECS/
--- nvi.spec 2010-06-22 14:29:33.000000000 +0200
+++ /home/michel/rpmbuild/SPECS/nvi.spec 2010-06-22 14:11:45.000000000 +0200
@@ -88,14 +88,14 @@
   vi_cv_revoke=no \
   ../dist/configure \
  --disable-curses \
- --prefix=%{_prefix} \
+ --prefix=/usr \
  --disable-shared --enable-static \
  --enable-widechar \
  --disable-threads \
  --without-x \
  --with-gnu-ld=yes \
- --datadir='%{_datadir}' \
- --mandir='%{_mandir}' \
+ --datadir='$${prefix}/share' \
+ --mandir='$${prefix}/share/man' \
  --program-prefix=n
 )
 sed -i -e '/define.*_PATH_MSGCAT/ s/".*"/"\/usr\/share\/vi\/catalog\/"/' \

-- 
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]