Opened 13 years ago

Closed 13 years ago

#30214 closed update (fixed)

kile-devel updated to stable version

Reported by: nicos_pavlov@… Owned by: NicosPavlov
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch maintainer Cc: pixilla (Bradley Giesbrecht)
Port: kile-devel

Description

kile for KDE 4 has reached a stable version recently. This update thus transfers kile-devel to a new stable port, and outdated kile-devel.

The name of the new port has been set to kile21 (kile name is taken by kile for KDE3), but some discussion has been going on to choose a correct name (see http://lists.macosforge.org/pipermail/macports-dev/2011-July/015114.html), so that it may be changed.

Note: the changes suggested for kile-devel in ticket #28400 are readily incorporated in the new port, enabling direct use of kile with okular, and possible integration with MacTeX version of TeXLive. All patch files are included also in this ticket.

Attachments (7)

Portfile-kile-devel.diff (1.1 KB) - added by nicos_pavlov@… 13 years ago.
Outdating current kile-devel port
patch-kilestdtools.rc.macports.diff (8.1 KB) - added by nicos_pavlov@… 13 years ago.
Updating paths for macports texlive
patch-kilestdtools.rc.texbin.diff (8.2 KB) - added by nicos_pavlov@… 13 years ago.
Updating paths for mactex texlive
patch-runTests.sh.macports.diff (305 bytes) - added by nicos_pavlov@… 13 years ago.
Correct runtests for macports texlive
patch-runTests.sh.texbin.diff (317 bytes) - added by nicos_pavlov@… 13 years ago.
Correct runtests for mactex texlive
Portfile (1.6 KB) - added by nicos_pavlov@… 13 years ago.
New kde4-kile port
Portfile-fulldependents (2.0 KB) - added by nicos_pavlov@… 13 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 13 years ago by mf2k (Frank Schima)

Keywords: haspatch maintainer added

Changed 13 years ago by nicos_pavlov@…

Attachment: Portfile-kile-devel.diff added

Outdating current kile-devel port

comment:2 Changed 13 years ago by nicos_pavlov@…

At this point, kile-devel is broken under KDE 4.7. This ticket thus corrects the present defect while passing to the new stable version at the same time.

Here is an update of the patch for the port, making it compatible with the update to KDE 4.7. The name of the new port has been changed to kde4-kile, in accordance with the new nomenclature kde4-runtime, for instance.

This ticket requires tickets #30493 (providing okular that kile requires, instead of kdegraphics4 which does not exist anymore) and ticket #30495 (providing kate which is not contained anymore in kdelibs4) to be committed before being usable.

comment:3 Changed 13 years ago by pixilla (Bradley Giesbrecht)

Nicolas: Your patches have many hard coded prefix paths that assume MacPorts is installed at /opt/local. Patching the paths to @PREFIX@ and using reinplace will work.

reinplace 's|@PREFIX@|${prefix}|g' filepath

See TCL Extensions reinplace here: http://guide.macports.org/#reference.tcl-extensions

Same path issue with /Applications.

reinplace 's|@APPLICATIONS_DIR@|${applications_dir}|g' filepath

See macports.conf for path variables here: http://guide.macports.org/#internals.configuration-files.macports-conf

In patch-kilestdtools.rc.texbin.diff you are linking to files in /usr and /usr/local. This should be avoided.

  • ImageMagick supplies ${prefix}/bin/convert so add a dep for ImageMagick.
  • Does texlive supply /texbin/* files? If so add a dep for textlive and use the same prefix treatment. If textlive does not supply texbin/bibtex and such we will want to find the port that does and add that port as a dep.

comment:4 Changed 13 years ago by pixilla (Bradley Giesbrecht)

Cc: pixilla@… added

Cc Me!

Changed 13 years ago by nicos_pavlov@…

Updating paths for macports texlive

Changed 13 years ago by nicos_pavlov@…

Updating paths for mactex texlive

Changed 13 years ago by nicos_pavlov@…

Correct runtests for macports texlive

Changed 13 years ago by nicos_pavlov@…

Correct runtests for mactex texlive

Changed 13 years ago by nicos_pavlov@…

Attachment: Portfile added

New kde4-kile port

Changed 13 years ago by nicos_pavlov@…

Attachment: Portfile-fulldependents added

comment:5 Changed 13 years ago by nicos_pavlov@…

Thanks for looking at it, and sorry for the prefix that I missed. I was indeed not sure about how to deal with them in a patch. This is corrected now. I tested it back, and it seems to work. All the dependencies on /usr/local have also been suppressed.

I would however disagree for adding a dependency on ImageMagick. The two files patched in this ticket are providing paths to kile for testing the availability of tools and providing corrects paths if available, which are not necessarily required, but just bring additional features. If all features (but not required) would have to be added, the following dependencies would have to be added as run dependencies: port:texlive-basic, port:texlive-latex, port:texlive-bin-extra (not for texbin variant), port:ImageMagick, port:ghostscript, port:kde4-baseapps

I nevertheless made another portfile (Portfile-fulldependents) which would contain the main dependencies for having full features in Kile.

About texbin path, it is indeed not provided by texlive port. By default, the *macports* patches are employed, linking to the texlive of macports (in ${prefix}). Then, I had the impression that the MacTeX installer for TeXLive was also tolerated with MacPorts, so that I added a texbin variants using the *texbin* patches, linking to the path where MacTeX installer puts the files (/usr/texbin/).

comment:6 Changed 13 years ago by NicosPavlov

Owner: changed from macports-tickets@… to nicos@…
Status: newassigned

comment:7 Changed 13 years ago by NicosPavlov

Resolution: fixed
Status: assignedclosed

Portfile kde4-kile added in r84698, with consistent dependencies (texlive, ghostscript and ImageMagick), and patches patch-kilestdtools.rc.macports.diff and patch-runTests.sh.macports.diff to add automatic connectivity to external tools of macports, solving ticket #28400. Texbin patches are not added at this point.

Portfile kile-devel made stub with replacement to kde4-kile in r84700.

Note: See TracTickets for help on using tickets.