Bug 206505 - [New port] science/mcxtrace a Monte: Carlo x-ray tracing package
Summary: [New port] science/mcxtrace a Monte: Carlo x-ray tracing package
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Yuri Victorovich
URL: https://reviews.freebsd.org/D13216
Keywords:
Depends on:
Blocks: 206506 206507
  Show dependency treegraph
 
Reported: 2016-01-22 20:32 UTC by Erik B Knudsen
Modified: 2017-12-05 19:15 UTC (History)
3 users (show)

See Also:
erkn: maintainer-feedback+


Attachments
shar archive conaining the actual port (5.27 KB, text/plain)
2016-01-22 20:39 UTC, Erik B Knudsen
rakuco: maintainer-approval+
Details
shar archive containing new version of the port the mcxtrace kernel (3.63 KB, text/plain)
2017-10-05 20:55 UTC, Erik B Knudsen
no flags Details
shar archive containing new version of the port the mcxtrace kernel (3.63 KB, text/plain)
2017-10-06 11:24 UTC, Erik B Knudsen
no flags Details
archive of port of the mcxtrace kernel latest (3.54 KB, application/x-shar)
2017-11-23 21:52 UTC, Erik B Knudsen
no flags Details
Updated shar (4.73 KB, text/plain)
2017-11-24 19:58 UTC, Yuri Victorovich
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Erik B Knudsen 2016-01-22 20:32:00 UTC
Monte Carlo X-ray tracing package for simulating x-ray scattering experiments.
This port contains the McXtrace kernel.
Comment 1 Erik B Knudsen 2016-01-22 20:39:02 UTC
Created attachment 165975 [details]
shar archive conaining the actual port

forgot to attach this in the original PR
Comment 2 Dmitry Marakasov freebsd_committer freebsd_triage 2016-03-27 23:10:33 UTC
- You should create links from port's Makefile and include them into plist as regular files. With that, you won't need install and deinstall scripts
- @dir entries are only needed for empty directories - you don't need @dir's for this port
- You should probably also link manpages
Comment 3 Erik B Knudsen 2017-10-05 20:55:57 UTC
Created attachment 186929 [details]
shar archive containing new version of the port the mcxtrace kernel

I've tested teh port using poudriere on 10.4 and 11.1 on i386 as well as amd64, and it installs OK. Some compiler warnings that need to be dealt with upstream.
Comment 4 Erik B Knudsen 2017-10-05 20:56:37 UTC
(In reply to Dmitry Marakasov from comment #2)
...sorry for taking this absurdly long to respond.
changes as sugegsted:
- links to executables created and added to pkg-plist
- install and deinstall scripts removed
- @dir entries in pkg-plist removed

The suggested links to man pages were not created done due to an upstream issue - which will be fixed in the next version.
Comment 5 Erik B Knudsen 2017-10-06 11:24:53 UTC
Created attachment 186948 [details]
shar archive containing new version of the port the mcxtrace kernel

Found a misprint in the URL
Comment 6 Yuri Victorovich freebsd_committer freebsd_triage 2017-11-04 21:38:21 UTC
Thank you for your submission.

These changes are needed:
* Please delete 'Created by' - these tags are obsolete
* Rename PORTVERSION to DISTVERSION
* LICENSE_FILE needs to be added
* License file is named LGPL, so the license is LGPLXX
* cmake needs to be changed to cmake:outsource (this is a default now)
* MANDIRS= needs a tab after it
* typo: designig -> designing


Thanks!
Yuri
Comment 7 Erik B Knudsen 2017-11-23 21:52:22 UTC
Created attachment 188226 [details]
archive of port of the mcxtrace kernel latest

this fixes the 6 things that yuri kindly mentioned
Comment 8 Erik B Knudsen 2017-11-23 21:53:05 UTC
(In reply to Yuri Victorovich from comment #6)
These things are now fixed in the latest veriosn of the port
Comment 9 Yuri Victorovich freebsd_committer freebsd_triage 2017-11-23 23:05:22 UTC
(In reply to Erik B Knudsen from comment #8)

Thank you for fixing them!
Comment 10 Yuri Victorovich freebsd_committer freebsd_triage 2017-11-24 19:58:26 UTC
Created attachment 188250 [details]
Updated shar

Why are .c/.h files installed?
Comment 11 Erik B Knudsen 2017-11-24 20:54:35 UTC
(In reply to Yuri Victorovich from comment #10)
This program works by generating (piecing together) a monolithic c-file for each specific scattering simulation. The .c and .h files are included bit by bit as needed - that's why they need to be present/installed. I can go more into detail if you want.
Comment 12 Yuri Victorovich freebsd_committer freebsd_triage 2017-11-24 20:56:16 UTC
(In reply to Erik B Knudsen from comment #11)

Thanks!
I just wanted to confirm if this isn't an accidental mistake.
Comment 13 Yuri Victorovich freebsd_committer freebsd_triage 2017-12-05 19:15:55 UTC
Committed with some changes.

Thank you for your contribution!
Comment 14 commit-hook freebsd_committer freebsd_triage 2017-12-05 19:15:59 UTC
A commit references this bug:

Author: yuri
Date: Tue Dec  5 19:15:50 UTC 2017
New revision: 455608
URL: https://svnweb.freebsd.org/changeset/ports/455608

Log:
  New port: science/mcxtrace: Monte Carlo X-ray tracing package

  PR:		206505
  Submitted by:	Erik B Knudsen <erkn@fysik.dtu.dk>
  Approved by:	tcberner (mentor)
  Differential Revision:	https://reviews.freebsd.org/D13216

Changes:
  head/science/Makefile
  head/science/mcxtrace/
  head/science/mcxtrace/Makefile
  head/science/mcxtrace/distinfo
  head/science/mcxtrace/files/
  head/science/mcxtrace/files/patch-cmake_Modules_InstallMCCODE.cmake
  head/science/mcxtrace/pkg-descr
  head/science/mcxtrace/pkg-plist