Picongpu: travis-ci suggestions only valid for GNU sed

Created on 17 Sep 2019  路  3Comments  路  Source: ComputationalRadiationPhysics/picongpu

The suggestions to automatically fix malformatted printed by the travis-ci dialog are great to quickly solve formating issues - but they are only valid if une used GNU sed. They cause various issues when using BSD sed, e.g.:

  • \t is only a tab in GNU, on BSD a real tab must be used (or alternatively the tr program)
  • 's/[[:blank:]]\+$//' does not remove trailing whitespaces
  • the flag -i requires a file extension

@ComputationalRadiationPhysics/picongpu-developers Do you think, we should additionally output equivalent commands for BSD?

tests tools

Most helpful comment

closed with #3073

All 3 comments

For the Windows version of GNU sed some of recommentations also do not work, while others do. Unfortunately, I do not remember which ones.

I just fixed the same in WarpX for you OSX users. I will share the updates with you :)

closed with #3073

Was this page helpful?
0 / 5 - 0 ratings

Related issues

bussmann picture bussmann  路  4Comments

berceanu picture berceanu  路  4Comments

cbontoiu picture cbontoiu  路  4Comments

berceanu picture berceanu  路  3Comments

ax3l picture ax3l  路  4Comments