Skip to content

first set of patches #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
vermaete wants to merge 9 commits into
base: master
Choose a base branch
from
Open

first set of patches #6

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
781705a
ci/cd: changed OpenDDS repo path from objectcomputing to OpenDDS
vermaete Oct 25, 2024
0887733
Building the 3.30 release of OpenDDS
vermaete Oct 25, 2024
b426f9a
changelog: fix Lintian warnings 'wrong-bug-number-in-closes' and 'cha…
vermaete Oct 25, 2024
b47246e
Use the default ace release of the OpenDDS configure script.
vermaete Oct 25, 2024
631761b
lintian: override the missing of the man pages
vermaete Oct 25, 2024
e9f11d6
github: changed the working directory from OpenDDS to opendss-VERSION
vermaete Oct 26, 2024
046df91
Lintian: removed pedantic (for now)
vermaete Oct 26, 2024
570719c
github action: allow the lintian step to fail.
vermaete Oct 26, 2024
458021f
github: using artifacts
vermaete Oct 26, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 14 additions & 6 deletions .github/workflows/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,22 +30,23 @@ jobs:
- name: Checkout OpenDDS
uses: actions/checkout@v4
with:
repository: 'objectcomputing/OpenDDS'
path: OpenDDS
repository: 'OpenDDS/OpenDDS'
path: opendds-3.30

@iguessthislldo iguessthislldo Oct 28, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why a hard-coded path like this?

@vermaete vermaete Oct 28, 2024

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@iguessthislldo It seems to be another rule of Debian. It was fixing this warning

dpkg-source: warning: source directory 'OpenDDS' is not - 'opendds-3.30'

And I assume it is to create the archive of the source code.

@vermaete vermaete Oct 28, 2024

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, I wasn't yet their to put the version into a variable.

ref: 'DDS-3.30'

- name: Copy debian Directory
run: cp -r OpenDDS-Debian/debian OpenDDS/debian
run: cp -r OpenDDS-Debian/debian opendds-3.30/debian

- name: Build Package
run: |
cd OpenDDS
cd opendds-3.30
dpkg-buildpackage -us -uc
cd ..
cp opendds_*.deb opendds.deb

- name: Commit New Deb
run: |
cd OpenDDS
cd opendds-3.30
SHA=$(git rev-parse --short HEAD)
cd ../OpenDDS-Debian
git config remote.origin.fetch "+refs/heads/*:refs/remotes/origin/*"
Expand All @@ -66,5 +67,12 @@ jobs:
branch: 'ubuntu_20_04'

- name: Lintian
run: lintian --pedantic --info --display-info opendds.deb
run: lintian --info --display-info opendds.deb
continue-on-error: true
if: always()

- uses: actions/upload-artifact@v4
with:
name: deb
path: opendds_*_*.deb
compression-level: 0 # no compression
8 changes: 7 additions & 1 deletion debian/changelog
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
opendds (3.30) UNRELEASED; urgency=medium

* Release 3.30

-- Fred Hornsey <hornseyf@objectcomputing.com> Thu, 24 Oct 2024 19:57:06 +0200

opendds (3.13-1) unstable; urgency=medium

* Initial release (Closes: #nnnn) <nnnn is the bug number of your ITP>
* Release 3.13

-- Fred Hornsey <hornseyf@objectcomputing.com> Sat, 19 Jan 2019 17:26:19 -0600
6 changes: 6 additions & 0 deletions debian/opendds.lintian-overrides
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
binary-without-manpage *

# ACE/TAO specific
national-encoding usr/share/ace/THANKS
national-encoding usr/share/tao/docs/releasenotes/notify.html
executable-not-elf-or-script usr/share/ace/bin/ace_install_pkgconfig.pl
2 changes: 1 addition & 1 deletion debian/rules
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,4 @@
dh $@

override_dh_auto_configure:
./configure --prefix=/usr --no-debug --optimize --no-tests --ace-github-latest
./configure --prefix=/usr --no-debug --optimize --no-tests