[pungi-fedora] PR #1397: Add -f option to enable syncing a second time

[Stephen Gallagher]

sgallagh opened a new pull-request against the project: `pungi-fedora` that you are following:
``
Add -f option to enable syncing a second time
``

To reply, visit the link below or just reply to this email
https://pagure.io/pungi-fedora/pull-request/1397

[Stephen Gallagher]

sgallagh commented on the pull-request: `Add -f option to enable syncing a second time` that you are following:
``
I'm working on a rewrite of https://docs.fedoraproject.org/en-US/eln/compose/ and while I was writing up the convoluted steps to modify LAST_COMPOSE_ID, I realized it would be easier to just enable the script to accept a `-f` argument.
``

To reply, visit the link below or just reply to this email
https://pagure.io/pungi-fedora/pull-request/1397

[Stephen Gallagher]

sgallagh commented on the pull-request: `Add -f option to enable syncing a second time` that you are following:
``
@ngompa Would you be willing to review this, please?
``

To reply, visit the link below or just reply to this email
https://pagure.io/pungi-fedora/pull-request/1397

[Neal Gompa]

ngompa commented on the pull-request: `Add -f option to enable syncing a second time` that you are following:
``
Does this actually work? Usually I have to split the conditional to use `||`, like `[[ $NEWCOMPOSE_DATE -gt $OLDCOMPOSE_DATE ]] || [[ -n "$FORCE_SYNC" ]]`
``

To reply, visit the link below or just reply to this email
https://pagure.io/pungi-fedora/pull-request/1397

[Stephen Gallagher]

sgallagh commented on the pull-request: `Add -f option to enable syncing a second time` that you are following:
``
```
% /usr/bin/cat conditional.sh 
#!/usr/bin/bash

OLD_DATE=2024
NEW_DATE=2024

string="nonzero"

if [[ $OLD_DATE -lt $NEW_DATE || -n "$string" ]]; then
  echo "Matched"
fi

% bash conditional.sh 
Matched
```
``

To reply, visit the link below or just reply to this email
https://pagure.io/pungi-fedora/pull-request/1397

[Neal Gompa]

ngompa commented on the pull-request: `Add -f option to enable syncing a second time` that you are following:
``
Okay I guess. :thumbsup: 
``

To reply, visit the link below or just reply to this email
https://pagure.io/pungi-fedora/pull-request/1397

[Stephen Gallagher]

sgallagh merged a pull-request against the project: `pungi-fedora` that you are following.

Merged pull-request:

``
Add -f option to enable syncing a second time
``

https://pagure.io/pungi-fedora/pull-request/1397