Remove missing %files in ldm.spec.in. Add runtime

Unidata / LDM

The Unidata Local Data Manager (LDM) system includes network client and server programs designed for event-driven data distribution, and is the fundamental component of the Unidata Internet Data Distribution (IDD) system.

http://www.unidata.ucar.edu/software/ldm

Other

43 stars 27 forks source link

Remove missing %files in ldm.spec.in. Add runtime #57

Closed NaderCHASER closed 7 years ago

NaderCHASER commented 7 years ago

Fixes “rpmbuild” errors caused by %files entries for files which no longer exist.
Adds “runtime” to %files to fix an error which prevented “rpmbuild” from completing.

semmerson commented 7 years ago

@NaderCHASER Because this is for an RPM distribution, I think it should include the NOAAPort stuff. Most user's won't use it -- but they can't unless it's there.

Can you modify what you've done to include the NOAAPort stuff, test it, and submit another pull request?

NaderCHASER commented 7 years ago

I'll see what I can do.

Despite the '--with-noaaport' configure option being in ldm.spec.in, my ~/rpmbuild/BUILDROOT/ does NOT contain NOAAPort docs under /share/doc/ldm/. This causes the "rpmbuild" to fail because the files are expected (%files) but not found.

NaderCHASER commented 7 years ago

My dish, which runs LDM 6.13.5, doesn't have the folder /share/doc/ldm/noaaport/ either. Can you check into this @semmerson?

semmerson commented 7 years ago

@NaderCHASER That directory should be /share/doc/ldm/utilities/noaaport/. Do a git pull and you should be OK.

NaderCHASER commented 7 years ago

Tested.

Wrote: /home/ldm/rpmbuild/RPMS/x86_64/ldm-6.13.5-1.el7.centos.x86_64.rpm Wrote: /home/ldm/rpmbuild/RPMS/x86_64/ldm-debuginfo-6.13.5-1.el7.centos.x86_64.rpm

semmerson commented 7 years ago

Excellent! Your changes have been merged.