https://bugzilla.redhat.com/show_bug.cgi?id=1662526
Ankur Sinha (FranciscoD) <sanjay.ankur(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |POST
--- Comment #10 from Ankur Sinha (FranciscoD) <sanjay.ankur(a)gmail.com> ---
(In reply to Dominik 'Rathann' Mierzejewski from comment #9)
Wouldn't it make sense to move
%{_includedir}/nrnconf.h
into
%{_includedir}/%{tarname}
I don't know where packages that will BuildRequires: neuron-devel expect
these files to be, so I'll leave this up to you to decide.
I've not seen too many refer to that header so I'll leave it for the time being
and change it later if required.
Also, I wouldn't define a macro that is longer than what it expands into,
i.e. I'd just use "nrn" everywhere instead of %{tarname}.
Ah, yes XD
None of the above are blockers and the package looks good otherwise, so
approved.
Thanks very much! SCM requested:
https://pagure.io/releng/fedora-scm-requests/issue/9737
Cheers,
Ankur
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component