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)
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 |
---|
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.
Changed 13 years ago by nicos_pavlov@…
Attachment: | patch-kilestdtools.rc.macports.diff added |
---|
Updating paths for macports texlive
Changed 13 years ago by nicos_pavlov@…
Attachment: | patch-kilestdtools.rc.texbin.diff added |
---|
Updating paths for mactex texlive
Changed 13 years ago by nicos_pavlov@…
Attachment: | patch-runTests.sh.macports.diff added |
---|
Correct runtests for macports texlive
Changed 13 years ago by nicos_pavlov@…
Attachment: | patch-runTests.sh.texbin.diff added |
---|
Correct runtests for mactex texlive
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: | new → assigned |
comment:7 Changed 13 years ago by NicosPavlov
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
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.
Outdating current kile-devel port